Closed
Bug 1017097
Opened 12 years ago
Closed 12 years ago
Autophone - support running local tests only
Categories
(Testing Graveyard :: Autophone, defect)
Testing Graveyard
Autophone
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bc, Assigned: bc)
Details
Attachments
(1 file)
|
2.23 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
As part of the move to Mountain View we will also need to at least temporarily only run the local tests. s1s2test.py expects that both the local and remote tests will always be run.
Need to allow this to be completely driven by the config file so that we can only local tests if we desire.
| Assignee | ||
Comment 1•12 years ago
|
||
Attachment #8431474 -
Flags: review?(mcote)
Comment 2•12 years ago
|
||
Comment on attachment 8431474 [details] [diff] [review]
bug-1017097.patch v1
Review of attachment 8431474 [details] [diff] [review]:
-----------------------------------------------------------------
::: tests/s1s2test.py
@@ +102,5 @@
> self._tests[t[0]] = t[1]
> # Map URLS - {urlname: url} - urlname serves as testname
> self._urls = {}
> + for test_location in cfg.options('locations'):
> + test_path = cfg.get('locations', test_location)
You can iterate over cfg.items('locations') to get the option name and value together and save a step.
Attachment #8431474 -
Flags: review?(mcote) → review+
| Assignee | ||
Comment 3•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•4 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•