Closed Bug 967566 Opened 10 years ago Closed 6 years ago

Refactor CommonTestCase.run

Categories

(Remote Protocol :: Marionette, defect, P3)

x86_64
Linux
defect

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.
Also tests.  Tests would be good.
Hi, I would like to work on this bug :)
Assignee: nobody → vikasmishra95
Whiteboard: [runner]
did this bug get lost?  I would like to ensure we keep moving forward if we can.
Assignee: vikasmishra95 → nobody
[mass update] Setting Harness bugs to all P3
Priority: -- → P3
Let’s not focus efforts on the harness.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.