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
|
||
Hi @richard67 ,
Is there an estimated timeline for merging this PR? Just wanted to plan accordingly.
Thank you!
Hi @richard67 , Is there an estimated timeline for merging this PR? Just wanted to plan accordingly. Thank you!
@D3S-Gaurav What do you have to plan? As every PR, this needs 2 successful human tests before it will be set to RTC (ready to commit) by a maintainer or bug squad member, and then it can be merged by @muhme or me, the release managers for 5.4.
@richard67 ,
Thank you for the clarification. I understand the process now and will wait for the required successful human tests before it can move to RTC.
Appreciate the guidance.
@D3S-Gaurav you don't need to open an issue if you are submitting a pull request