Merge watchlistloader into host since it's an unnecessary indirection
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 13 Apr 2013 09:47:21 +0000 (09:47 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 13 Apr 2013 09:47:21 +0000 (09:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114555

Reviewed by Antti Koivisto.

We don't need a class to load a file.

* Scripts/webkitpy/common/host.py:
(Host.watch_list):
* Scripts/webkitpy/common/watchlist/watchlistloader.py: Removed.
* Scripts/webkitpy/common/watchlist/watchlistloader_unittest.py: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@148354 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/common/host.py
Tools/Scripts/webkitpy/common/watchlist/watchlistloader.py [deleted file]
Tools/Scripts/webkitpy/common/watchlist/watchlistloader_unittest.py [deleted file]

index 607880c06ac7686e9338de9b5a6dadcbfe2a268e..4e510f3a07f9a1241e807543262edcb534e06e41 100644 (file)
@@ -1,3 +1,17 @@
+2013-04-13  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Merge watchlistloader into host since it's an unnecessary indirection
+        https://bugs.webkit.org/show_bug.cgi?id=114555
+
+        Reviewed by Antti Koivisto.
+
+        We don't need a class to load a file.
+
+        * Scripts/webkitpy/common/host.py:
+        (Host.watch_list):
+        * Scripts/webkitpy/common/watchlist/watchlistloader.py: Removed.
+        * Scripts/webkitpy/common/watchlist/watchlistloader_unittest.py: Removed.
+
 2013-04-13  Ryosuke Niwa  <rniwa@webkit.org>
 
         Rename JSONResultsGeneratorBase to JSONResultsGenerator
index c7ba15e905a26ab7e7f40c588222bd5a2a5c547e..ef4b7ef8366c9660b84d4995ee65fec3b0288015 100644 (file)
@@ -36,7 +36,7 @@ from webkitpy.common.checkout.scm.detection import SCMDetector
 from webkitpy.common.memoized import memoized
 from webkitpy.common.net import bugzilla, buildbot, web
 from webkitpy.common.system.systemhost import SystemHost
-from webkitpy.common.watchlist.watchlistloader import WatchListLoader
+from webkitpy.common.watchlist.watchlistparser import WatchListParser
 from webkitpy.layout_tests.port.factory import PortFactory
 
 
@@ -91,4 +91,10 @@ class Host(SystemHost):
 
     @memoized
     def watch_list(self):
-        return WatchListLoader(self.filesystem).load()
+        config_path = self.filesystem.dirname(self.filesystem.path_to_module('webkitpy.common.config'))
+        watch_list_full_path = self.filesystem.join(config_path, 'watchlist')
+        if not self.filesystem.exists(watch_list_full_path):
+            raise Exception('Watch list file (%s) not found.' % watch_list_full_path)
+
+        watch_list_contents = self.filesystem.read_text_file(watch_list_full_path)
+        return WatchListParser().parse(watch_list_contents)
diff --git a/Tools/Scripts/webkitpy/common/watchlist/watchlistloader.py b/Tools/Scripts/webkitpy/common/watchlist/watchlistloader.py
deleted file mode 100644 (file)
index aa816e3..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-# Copyright (C) 2011 Google Inc. All rights reserved.
-#
-# Redistribution and use in source and binary forms, with or without
-# modification, are permitted provided that the following conditions are
-# met:
-#
-#     * Redistributions of source code must retain the above copyright
-# notice, this list of conditions and the following disclaimer.
-#     * Redistributions in binary form must reproduce the above
-# copyright notice, this list of conditions and the following disclaimer
-# in the documentation and/or other materials provided with the
-# distribution.
-#     * Neither the name of Google Inc. nor the names of its
-# contributors may be used to endorse or promote products derived from
-# this software without specific prior written permission.
-#
-# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
-# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
-# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
-# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
-# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
-# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
-# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
-# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-from webkitpy.common.watchlist.watchlistparser import WatchListParser
-
-
-class WatchListLoader(object):
-    def __init__(self, filesystem):
-        self._filesystem = filesystem
-
-    def load(self):
-        config_path = self._filesystem.dirname(self._filesystem.path_to_module('webkitpy.common.config'))
-        watch_list_full_path = self._filesystem.join(config_path, 'watchlist')
-        if not self._filesystem.exists(watch_list_full_path):
-            raise Exception('Watch list file (%s) not found.' % watch_list_full_path)
-
-        watch_list_contents = self._filesystem.read_text_file(watch_list_full_path)
-        return WatchListParser().parse(watch_list_contents)
diff --git a/Tools/Scripts/webkitpy/common/watchlist/watchlistloader_unittest.py b/Tools/Scripts/webkitpy/common/watchlist/watchlistloader_unittest.py
deleted file mode 100644 (file)
index 8d3fa98..0000000
+++ /dev/null
@@ -1,45 +0,0 @@
-# Copyright (C) 2011 Google Inc. All rights reserved.
-#
-# Redistribution and use in source and binary forms, with or without
-# modification, are permitted provided that the following conditions are
-# met:
-#
-#    * Redistributions of source code must retain the above copyright
-# notice, this list of conditions and the following disclaimer.
-#    * Redistributions in binary form must reproduce the above
-# copyright notice, this list of conditions and the following disclaimer
-# in the documentation and/or other materials provided with the
-# distribution.
-#    * Neither the name of Google Inc. nor the names of its
-# contributors may be used to endorse or promote products derived from
-# this software without specific prior written permission.
-#
-# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
-# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
-# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
-# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
-# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
-# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
-# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
-# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-'''Unit tests for watchlistloader.py.'''
-
-from webkitpy.common import webkitunittest
-from webkitpy.common.system import filesystem_mock
-from webkitpy.common.system import filesystem
-from webkitpy.common.system.outputcapture import OutputCapture
-from webkitpy.common.watchlist.watchlistloader import WatchListLoader
-
-
-class WatchListLoaderTest(webkitunittest.TestCase):
-    def test_watch_list_not_found(self):
-        loader = WatchListLoader(filesystem_mock.MockFileSystem())
-        self.assertRaisesRegexp(Exception, r'Watch list file \(.*/watchlist\) not found\.', loader.load)
-
-    def test_watch_list_load(self):
-        # Test parsing of the checked-in watch list.
-        OutputCapture().assert_outputs(self, WatchListLoader(filesystem.FileSystem()).load, expected_logs="")