Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(829)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool_unittest.py

Issue 2070333003: Use an explicit list of webkit-patch commands instead of using auto-discovery. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool_unittest.py
index 10d9ede0d6ea0c4d32103d9dda2b00857f445367..9b003e431fa03583a34dd18997fee4fd20927aa5 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool_unittest.py
@@ -32,7 +32,7 @@ from optparse import make_option
from webkitpy.common.system.outputcapture import OutputCapture
from webkitpy.tool.multi_command_tool import MultiCommandTool
-from webkitpy.tool.commands import Command
+from webkitpy.tool.commands.command import Command
class TrivialCommand(Command):
@@ -85,8 +85,7 @@ class MultiCommandToolTest(unittest.TestCase):
self._assert_split(full_args, full_args_expected)
def test_command_by_name(self):
- # This also tests Command auto-discovery.
- tool = TrivialTool()
+ tool = TrivialTool(commands=[TrivialCommand(), UncommonCommand()])
self.assertEqual(tool.command_by_name("trivial").name, "trivial")
self.assertIsNone(tool.command_by_name("bar"))

Powered by Google App Engine
This is Rietveld 408576698