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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/common/system/system_host_mock.py

Issue 2588863002: Make MockExecutive behave more like MockExecutive2. (Closed)
Patch Set: Reformat args lists for MockHost, MockSystemHost. 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/common/system/system_host_mock.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/common/system/system_host_mock.py b/third_party/WebKit/Tools/Scripts/webkitpy/common/system/system_host_mock.py
index 1bff4c6cb74163cdb612fdecfe142eb5de8b5157..fcdd1e8aa3b7476b75c561047fff4c2c8ada3bfd 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/common/system/system_host_mock.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/common/system/system_host_mock.py
@@ -37,10 +37,15 @@ from webkitpy.common.system.workspace_mock import MockWorkspace
class MockSystemHost(object):
- def __init__(self, log_executive=False, executive_throws_when_run=None, os_name=None,
- os_version=None, executive=None, filesystem=None, time_return_val=123):
+ def __init__(self,
+ log_executive=False,
+ os_name=None,
+ os_version=None,
+ executive=None,
+ filesystem=None,
+ time_return_val=123):
self.executable = 'python'
- self.executive = executive or MockExecutive(should_log=log_executive, should_throw_when_run=executive_throws_when_run)
+ self.executive = executive or MockExecutive(should_log=log_executive)
self.filesystem = filesystem or MockFileSystem()
self.user = MockUser()
self.platform = MockPlatformInfo()

Powered by Google App Engine
This is Rietveld 408576698