[FEATURE] Speed up Cypress E2E test suite (3m31s → 58s)

[Sourya07]

Why do we need this improvement?

The Cypress E2E suite was slow due to:

1. Repeated cy.visit() calls
2. Overuse of beforeEach() for read-only tests
3. A skipped test caused by a nested it block
4. Retries enabled during local runs

How will this change help?

1. toolspage.cy.js: Replaced beforeEach() with before() (page loaded once instead of 57 times)
   ~1m52s → ~1s
2. homepage.cy.js: Split navigation tests from verification tests ~47s → ~28s
3. docspage.cy.js: Fixed invalid nested it block so all tests run
4. Config:
5. Set testIsolation: false
6. Added test:e2e:fast (retries=0) for quick local feedback

Screenshots

No response

How could it be implemented/designed?

1. Reduce redundant reloads: Use before() instead of beforeEach() for read-only tests and share page state by disabling test isolation where safe.

2. Optimize test structure: Separate navigation tests from verification-only tests and avoid unnecessary external navigation (assert href instead).

🚧 Breaking changes

Yes

👀 Have you checked for similar open issues?

• I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to work on this issue?

Yes I am willing to submit a PR!

[princerajpoot20]

Repeated cy.visit() calls

Umm… this does not make sense to me. It’s like saying we should remove test cases to make them run faster. We are using it for a purpose. If you think we are using visit() unnecessarily, then point it out specifically instead of saying it in a generic way.

Also, can you provide some more concrete references so that we can take a decision more quickly on what needs to be done and what does not

[Sourya07]

Thanks for the feedback @princerajpoot20!
I am not removing any test cases. All 78 tests still run and verify the exact same things.

The optimization addresses how the test runner sets up the page for read-only tests.

The Specific Bottleneck

for now we have
toolspage.cy.js
iterates through ~57 tools. For each tool, it runs beforeEach(() => cy.visit('/tools')).

This means we are reloading the entire React application 57 times just to check if a link exists on the page.
Cost: ~1 minute 52 seconds.
The Solution
before(() => cy.visit('/tools')) t.oolsData.forEach(tool => { it(verifies ${tool.name}, ...) })
I have changed beforeEach to before for this file.

We visit /tools once.
We run the exact same 57 assertions against that single page load.
Since these tests effectively only read the DOM and do not mutate state, it is safe to reuse the page provided we disable testIsolation for these specific read-only suites.

New Cost: ~1 second.

1. We are checking for the presence of links and text.
2. Test A checking checking "Tool X" does not change the state for Test B checking "Tool Y".
3. For Navigation tests (where clicking a link changes the URL), we keep beforeEach
   I applied similar logic to the static verification parts of ->

homepage.cy.js

to ensure isolation.
.

Result
Coverage remains 100% identical, but the suite now finishes in 58s instead of 3m 31s.

I hope this provides the concrete context you were looking for! 👍
looking forward for this
Thank you 🙏🏻

[Sourya07]

overall its increase the performance

before it was taking this much time
after implementation this takes ->

[Sourya07]

@princerajpoot20 please look on this .