Closed Bug 1141951 Opened 10 years ago Closed 7 years ago

BaseMarionetteOptions.verify_usage() should use self.error() instead of print and sys.exit(1)

Categories

(Remote Protocol :: Marionette, defect)

Product:
Remote Protocol
Component:
Marionette
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1258505

People

(Reporter: whimboo, Unassigned)

References

(
URL
)

Details

The option parser has a method called `error()` which can be used to print out errors to the console and exit the application. That means instead of: > if not tests: > print 'must specify one or more test files, manifests, or directories' > sys.exit(1) we can use: > if not tests: > self.error('must specify one or more test files, manifests, or directories') There are a couple of those instances left, which would need a fix.
BaseMarionetteOptions does not appear to exist anymore.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Fixed as part of bug 1258505.
Resolution: INVALID → DUPLICATE
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.