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

Unified Diff: tools/telemetry/telemetry/benchmark.py

Issue 337603005: Rename telemetry.test.Test to telemetry.benchmark.Benchmark. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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: tools/telemetry/telemetry/benchmark.py
diff --git a/tools/telemetry/telemetry/test.py b/tools/telemetry/telemetry/benchmark.py
similarity index 96%
rename from tools/telemetry/telemetry/test.py
rename to tools/telemetry/telemetry/benchmark.py
index c89ba8ef8635d219c45d23c788ee7ab414dda9aa..d0dd279dd6a7c3a45e3c87a971d0398c25e9841e 100644
--- a/tools/telemetry/telemetry/test.py
+++ b/tools/telemetry/telemetry/benchmark.py
@@ -25,8 +25,8 @@ Disabled = decorators.Disabled
Enabled = decorators.Enabled
-class Test(command_line.Command):
- """Base class for a Telemetry test or benchmark.
+class Benchmark(command_line.Command):
+ """Base class for a Telemetry benchmark.
A test packages a PageTest/PageMeasurement and a PageSet together.
"""
@@ -157,9 +157,9 @@ class Test(command_line.Command):
@classmethod
def PageTestClass(cls):
- """Get the PageTest for this Test.
+ """Get the PageTest for this Benchmark.
- If the Test has no PageTest, raises NotImplementedError.
+ If the Benchmark has no PageTest, raises NotImplementedError.
"""
if not hasattr(cls, 'test'):
raise NotImplementedError('This test has no "test" attribute.')
@@ -169,9 +169,9 @@ class Test(command_line.Command):
@classmethod
def PageSetClass(cls):
- """Get the PageSet for this Test.
+ """Get the PageSet for this Benchmark.
- If the Test has no PageSet, raises NotImplementedError.
+ If the Benchmark has no PageSet, raises NotImplementedError.
"""
if not hasattr(cls, 'page_set'):
raise NotImplementedError('This test has no "page_set" attribute.')

Powered by Google App Engine
This is Rietveld 408576698