Closed Bug 1141951 Opened 9 years ago Closed 6 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: 6 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.