Browse Source

Check for browsers in verifyOptions, so the script fails early.

Rob Sayre 14 years ago
parent
commit
605535e94e
  1. 5
      test/test.py

5
test/test.py

@ -37,6 +37,8 @@ class TestOptions(OptionParser):
options.manifestFile = DEFAULT_MANIFEST_FILE options.manifestFile = DEFAULT_MANIFEST_FILE
if options.browser and options.browserManifestFile: if options.browser and options.browserManifestFile:
print "Warning: ignoring browser argument since manifest file was also supplied" print "Warning: ignoring browser argument since manifest file was also supplied"
if not options.browser and not options.browserManifestFile:
self.error("No test browsers found. Use --browserManifest or --browser args.")
return options return options
def prompt(question): def prompt(question):
@ -220,8 +222,7 @@ def setUp(options):
testBrowsers = makeBrowserCommands(options.browserManifestFile) testBrowsers = makeBrowserCommands(options.browserManifestFile)
elif options.browser: elif options.browser:
testBrowsers = [BrowserCommand({"path":options.browser, "name":"firefox"})] testBrowsers = [BrowserCommand({"path":options.browser, "name":"firefox"})]
else: assert len(testBrowsers) > 0
print "No test browsers found. Use --browserManifest or --browser args."
with open(options.manifestFile) as mf: with open(options.manifestFile) as mf:
manifestList = json.load(mf) manifestList = json.load(mf)

Loading…
Cancel
Save