diff options
| author | Stefan Giehl <stefan@matomo.org> | 2021-09-01 16:37:38 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-01 16:37:38 +0300 |
| commit | a3ee690ea3863ff16a828ac0afbc77243bb45f5b (patch) | |
| tree | 77f3341e86608894b1020725700b348bbcd44567 /plugins/Morpheus | |
| parent | ba1facfd1ecb4e618f3162aedbdb3170e9461e50 (diff) | |
Use newer version of Puppeteer for UI tests (#17880)
* Use latest version of Puppeteer for UI tests
* use newer node on travis
* test
* use waitForTimeout / waitForSelector instead of waitFor
* Handle BC for removed waitFor method
* Updates expected UI test files
* update submodules
* improve/fix some UI tests
* Updates expected UI test files
* improve/fix some UI tests
* update url-parse dependency
* simplify some UI tests
* update screenshot
* avoid using merged assets that are broken
* fix expected compare exit code
* don't use merged assets for TwoFactorAuth UI tests
it seems due to too many requests to login, logout and other pages merging the CoreJS assets somehow fails, and results in returning an empty file. Even the reload fails to serve the correct content, so maybe the file, the content should be written to, is somehow locked. Disabling the merged assets seems to fix this issue for this tests. Locally the tests are also running correctly with merged assets.
* improve handling of automatic css / js reloading in UI tests
* increase timeouts of TwoFactorAuth UI tests
* fix 2fa ui test
* revert change
* update new screenshot
* updates UI screenshots
* submodule updates
Diffstat (limited to 'plugins/Morpheus')
| -rw-r--r-- | plugins/Morpheus/tests/UI/Morpheus_spec.js | 4 | ||||
| -rw-r--r-- | plugins/Morpheus/tests/UI/expected-screenshots/Morpheus_load.png | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/plugins/Morpheus/tests/UI/Morpheus_spec.js b/plugins/Morpheus/tests/UI/Morpheus_spec.js index a6685d85ce..44f5de9302 100644 --- a/plugins/Morpheus/tests/UI/Morpheus_spec.js +++ b/plugins/Morpheus/tests/UI/Morpheus_spec.js @@ -18,13 +18,13 @@ describe("Morpheus", function () { it("should show all UI components and CSS classes", async function() { await page.goto(url); - await page.waitFor('.progressbar img'); + await page.waitForSelector('.progressbar img'); await page.evaluate(() => { $('img[src~=loading],.progressbar img').each(function () { $(this).hide(); }); }); - await page.waitFor(500); // wait for rendering + await page.waitForTimeout(500); // wait for rendering expect(await page.screenshot({ fullPage: true })).to.matchImage('load'); }); }); diff --git a/plugins/Morpheus/tests/UI/expected-screenshots/Morpheus_load.png b/plugins/Morpheus/tests/UI/expected-screenshots/Morpheus_load.png index 9658f9a389..e9f27f7dff 100644 --- a/plugins/Morpheus/tests/UI/expected-screenshots/Morpheus_load.png +++ b/plugins/Morpheus/tests/UI/expected-screenshots/Morpheus_load.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:5aac1670a60d3d05918ac70af0f6f2fd519225a75b0c0f5ecdd4651532c77b59 -size 1669625 +oid sha256:194ae49ce59d719738177a06b90df92805753936160f23e5e12a32904ff20c47 +size 1670689 |
