User tests: Successful: Unsuccessful:
Pull Request resolves #46931 .
Added explicit { timeout: 30000 } (30 seconds) to both cy.contains() assertions in the FromUrl.cy.js system test. The test downloads and installs a real extension from a live GitHub URL, which can take significantly longer than Cypress's default 4-second timeout. This makes the test flaky in CI environments with variable network latency.
Changes:
cy.contains('Installation of the component was successful.', { timeout: 30000 });cy.contains('Uninstalling the component was successful', { timeout: 30000 });"can install and uninstall a component from URL tab" passes reliably.The test intermittently fails with:
AssertionError: Timed out retrying after 4000ms: Expected to find content: 'Installation of the component was successful.'
This happens because the default Cypress timeout (4s) is insufficient for downloading and installing an extension from an external URL.
The test reliably waits up to 30 seconds for the installation and uninstallation to complete, eliminating flaky failures caused by network latency.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | JavaScript Unit Tests |
| Title |
|
||||||
| Labels |
Added:
Unit/System Tests
bug
PR-5.4-dev
|
||
@D3S-Gaurav you don't need to open an issue if you are submitting a pull request