Commit ee411a73 authored by Jérome Perrin's avatar Jérome Perrin

fixup! fixup: ERP5TypeFunctionalTestCase: support running tests on seleniumserver

we only want to set options to None when not using firefox, but we use firefox
by default. See dicussion on nexedi/erp5@17e1c262 (comment 136933)
parent 17e1c262
...@@ -241,7 +241,7 @@ class FunctionalTestRunner: ...@@ -241,7 +241,7 @@ class FunctionalTestRunner:
test_runner_configuration = json.load(f) test_runner_configuration = json.load(f)
selenium_test_runner_configuration = test_runner_configuration.get('selenium', {}) selenium_test_runner_configuration = test_runner_configuration.get('selenium', {})
use_local_firefox = selenium_test_runner_configuration.get('server-url') is None use_local_firefox = selenium_test_runner_configuration.get('server-url') is None
if 'firefox' not in selenium_test_runner_configuration.get( if not use_local_firefox and 'firefox' not in selenium_test_runner_configuration.get(
'desired-capabilities', {}).get('browserName', '').lower(): 'desired-capabilities', {}).get('browserName', '').lower():
options = None options = None
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment