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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py

Issue 2605873004: Unify MockExecutive and MockExecutive2. (Closed)
Patch Set: Created 4 years 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/w3c/deps_updater_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
index 57c6b766261fe68ad0e9b94c42cc9690567c4021..806ca41ed9770547de2b96b9eaae11e299f28c44 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
@@ -4,9 +4,9 @@
import unittest
-from webkitpy.w3c.deps_updater import DepsUpdater
from webkitpy.common.host_mock import MockHost
-from webkitpy.common.system.executive_mock import MockExecutive2
+from webkitpy.common.system.executive_mock import MockExecutive
+from webkitpy.w3c.deps_updater import DepsUpdater
class DepsUpdaterTest(unittest.TestCase):
@@ -64,7 +64,7 @@ class DepsUpdaterTest(unittest.TestCase):
def test_cl_description_with_empty_environ(self):
host = MockHost()
- host.executive = MockExecutive2(output='Last commit message\n')
+ host.executive = MockExecutive(output='Last commit message\n')
updater = DepsUpdater(host)
description = updater._cl_description()
self.assertEqual(
@@ -76,7 +76,7 @@ class DepsUpdaterTest(unittest.TestCase):
def test_cl_description_with_environ_variables(self):
host = MockHost()
- host.executive = MockExecutive2(output='Last commit message\n')
+ host.executive = MockExecutive(output='Last commit message\n')
updater = DepsUpdater(host)
updater.host.environ['BUILDBOT_MASTERNAME'] = 'my.master'
updater.host.environ['BUILDBOT_BUILDERNAME'] = 'b'

Powered by Google App Engine
This is Rietveld 408576698