Closed
Bug 967566
Opened 11 years ago
Closed 7 years ago
Refactor CommonTestCase.run
Categories
(Remote Protocol :: Marionette, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ato, Unassigned)
Details
(Keywords: pi-marionette-runner, Whiteboard: [runner])
The CommonTestCase.run function in testing/marionette/client/marionette/marionette_test.py:101 deserves a refactor to reduce code cyclomatic complexity, code duplication, and generally to make it easier to understand.
Also I'm not sure that the current approach of using exceptions for control the code flow is the best design for this class.
|
Reporter | |
Comment 1•11 years ago
|
||
Also tests. Tests would be good.
|
||
Comment 2•11 years ago
|
||
Hi, I would like to work on this bug :)
|
||
Updated•11 years ago
|
Assignee: nobody → vikasmishra95
|
||
Updated•11 years ago
|
Whiteboard: [runner]
|
|
||
Comment 3•11 years ago
|
||
did this bug get lost? I would like to ensure we keep moving forward if we can.
|
|
||
Updated•10 years ago
|
Keywords: ateam-marionette-runner
|
|
||
Updated•9 years ago
|
Assignee: vikasmishra95 → nobody
|
|
Reporter | |
Comment 5•7 years ago
|
||
Let’s not focus efforts on the harness.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
|
||
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•