Automated tests: improve handing of missing .xpi
Background
Firefox support was added to our E2E automation testsuite in https://gitlab.com/eyeo/adblockplus/abpui/adblockplusui/-/issues/1100.
There is a function to check if the .xpi
file exists before the tests are run (see helpers.js
, async function beforeSequence()
), but it does not work as intended- at least on Linux:
- does not find the
.xpi
even if it exists - throws an error, but does not stop the execution if the file is not found
- the check is performed before each test, instead of only once, before the browsers are started
What to change
- Design: N/A
- Research: N/A
- Spec: N/A
- Legal: N/A
-
Development:
- figure out why it doesn't find the
.xpi
when running the tests on linux - if the
.xpi
is missing, the execution should stop- the same as it does if thedevenv.chrome
is missing for running the tests on Chrome - change the check so it's run once, before running the tests
- figure out why it doesn't find the
Hints for testers
None.
Hints for translators
N/A
Edited by Thomas Greiner