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

Unified Diff: tools/telemetry/telemetry/page/record_wpr_unittest.py

Issue 637153002: telemetry: Remove command line args from page test (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 6 years, 2 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: tools/telemetry/telemetry/page/record_wpr_unittest.py
diff --git a/tools/telemetry/telemetry/page/record_wpr_unittest.py b/tools/telemetry/telemetry/page/record_wpr_unittest.py
index 75e79b2adaa6ccf4d086cf9511ea6a41e9293bb6..13fd8868e9053bc5dbe0af9d3b3b6466d4b7ce10 100644
--- a/tools/telemetry/telemetry/page/record_wpr_unittest.py
+++ b/tools/telemetry/telemetry/page/record_wpr_unittest.py
@@ -49,10 +49,6 @@ class MockPageTest(page_test.PageTest):
self._action_name_to_run = "RunBaz"
self.func_calls = []
- @classmethod
- def AddCommandLineArgs(cls, parser):
- parser.add_option('--mock-page-test-option', action="store_true")
-
def WillNavigateToPage(self, page, tab):
self.func_calls.append('WillNavigateToPage')
@@ -167,7 +163,6 @@ class RecordWprUnitTests(tab_test_case.TabTestCase):
flags = [
'--page-repeat', '2',
'--mock-benchmark-url', self._url,
- '--mock-page-test-option',
]
wpr_recorder = record_wpr.WprRecorder(self._test_data_dir, MockBenchmark(),
flags)
@@ -175,8 +170,6 @@ class RecordWprUnitTests(tab_test_case.TabTestCase):
self.assertEquals(2, wpr_recorder.options.page_repeat)
# benchmark command-line args
self.assertEquals(self._url, wpr_recorder.options.mock_benchmark_url)
- # benchmark's page_test command-line args
- self.assertTrue(wpr_recorder.options.mock_page_test_option)
# invalid command-line args
self.assertFalse(hasattr(wpr_recorder.options, 'not_a_real_option'))

Powered by Google App Engine
This is Rietveld 408576698