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

Unified Diff: tools/telemetry/telemetry/story/story_set.py

Issue 1126443002: Move user_story.user_story_set to story.story_set (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 7 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/story/story_set.py
diff --git a/tools/telemetry/telemetry/user_story/user_story_set.py b/tools/telemetry/telemetry/story/story_set.py
similarity index 88%
rename from tools/telemetry/telemetry/user_story/user_story_set.py
rename to tools/telemetry/telemetry/story/story_set.py
index d18941cccca675cdf16feb9340268effb4876b89..3a2c7c7869b09dc19ce40a2bb87a6bb6fcaff627 100644
--- a/tools/telemetry/telemetry/user_story/user_story_set.py
+++ b/tools/telemetry/telemetry/story/story_set.py
@@ -6,32 +6,27 @@ import inspect
import os
from telemetry import user_story as user_story_module
-from telemetry.util import cloud_storage
from telemetry.wpr import archive_info
-PUBLIC_BUCKET = cloud_storage.PUBLIC_BUCKET
-PARTNER_BUCKET = cloud_storage.PARTNER_BUCKET
-INTERNAL_BUCKET = cloud_storage.INTERNAL_BUCKET
-
-class UserStorySet(object):
+class StorySet(object):
"""A collection of user story.
- A typical usage of UserStorySet would be to subclass it and then calling
+ A typical usage of StorySet would be to subclass it and then calling
AddUserStory for each UserStory.
"""
def __init__(self, archive_data_file='', cloud_storage_bucket=None,
base_dir=None, serving_dirs=None):
- """Creates a new UserStorySet.
+ """Creates a new StorySet.
Args:
archive_data_file: The path to Web Page Replay's archive data, relative
to self.base_dir.
cloud_storage_bucket: The cloud storage bucket used to download
Web Page Replay's archive data. Valid values are: None,
- PUBLIC_BUCKET, PARTNER_BUCKET, or INTERNAL_BUCKET (defined
- in telemetry.util.cloud_storage).
+ story.PUBLIC_BUCKET, story.PARTNER_BUCKET, or story.INTERNAL_BUCKET
+ (defined in telemetry.util.cloud_storage).
serving_dirs: A set of paths, relative to self.base_dir, to directories
containing hash files for non-wpr archive data stored in cloud
storage.
@@ -56,7 +51,7 @@ class UserStorySet(object):
"""True iff UserStories are allowed to have different StoryState classes.
There are no checks in place for determining if SharedStates are
- being assigned correctly to all UserStorys in a given UserStorySet. The
+ being assigned correctly to all UserStorys in a given StorySet. The
majority of test cases should not need the ability to have multiple
ShareduserStoryStates, and usually implies you should be writing multiple
benchmarks instead. We provide errors to avoid accidentally assigning
@@ -73,7 +68,7 @@ class UserStorySet(object):
def base_dir(self):
"""The base directory to resolve archive_data_file.
- This defaults to the directory containing the UserStorySet instance's class.
+ This defaults to the directory containing the StorySet instance's class.
"""
return self._base_dir
@@ -114,7 +109,7 @@ class UserStorySet(object):
@classmethod
def Name(cls):
- """Returns the string name of this UserStorySet.
+ """Returns the string name of this StorySet.
Note that this should be a classmethod so benchmark_runner script can match
user story class with its name specified in the run command:
'Run <User story test name> <User story class name>'
« no previous file with comments | « tools/telemetry/telemetry/story/story_filter.py ('k') | tools/telemetry/telemetry/story/story_set_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698