Unit/System Tests bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar D3S-Gaurav
D3S-Gaurav
20 Feb 2026

Pull Request resolves #46931 .

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

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:

  • Line 18: cy.contains('Installation of the component was successful.', { timeout: 30000 });
  • Line 28: cy.contains('Uninstalling the component was successful', { timeout: 30000 });

Testing Instructions

  1. Run the system test for the installer component:
    npx cypress run --spec tests/System/integration/administrator/components/com_installer/FromUrl.cy.js
  2. Verify the test "can install and uninstall a component from URL tab" passes reliably.
  3. Optionally simulate a slow network to confirm the increased timeout prevents false failures.

Actual result BEFORE applying this Pull Request

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.

Expected result AFTER applying this Pull Request

The test reliably waits up to 30 seconds for the installation and uninstallation to complete, eliminating flaky failures caused by network latency.

Link to documentations

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

avatar D3S-Gaurav D3S-Gaurav - open - 20 Feb 2026
avatar D3S-Gaurav D3S-Gaurav - change - 20 Feb 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Feb 2026
Category JavaScript Unit Tests
avatar alikon
alikon - comment - 21 Feb 2026

@D3S-Gaurav you don't need to open an issue if you are submitting a pull request

avatar D3S-Gaurav
D3S-Gaurav - comment - 21 Feb 2026

Hi @alikon ,
Thanks for the information. Also should I close the issue.

avatar alikon
alikon - comment - 21 Feb 2026

as per #46835
Successfully merging this pull request may close these issues.

avatar richard67 richard67 - change - 21 Feb 2026
Title
Fix flaky FromUrl test by adding explicit timeout for network-depende…
[5.4] Fix flaky FromUrl test by adding explicit timeout for network-depende…
avatar richard67 richard67 - edited - 21 Feb 2026
avatar D3S-Gaurav D3S-Gaurav - change - 22 Feb 2026
Labels Added: Unit/System Tests bug PR-5.4-dev
avatar D3S-Gaurav
D3S-Gaurav - comment - 25 Feb 2026

Hi @richard67 ,
Is there an estimated timeline for merging this PR? Just wanted to plan accordingly.
Thank you!

avatar richard67
richard67 - comment - 25 Feb 2026

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.

avatar D3S-Gaurav
D3S-Gaurav - comment - 25 Feb 2026

@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.

Add a Comment

Login with GitHub to post a comment