Improved page loading in testing

[LeandroPata]

Improved page loading in testing

Summary

Updated the test suite to use domcontentloaded instead of networkidle, when applicable, for better control over when tests execute. This ensures that tests run when the DOM is fully loaded, improving reliability and performance. Also added test fixtures to avoid tests being network dependent whenever possible.

Changes

• Updated wait strategy in multiple tests:

  • In test/E2ETesting/gallery.spec.ts, updated the page.goto call to use { waitUntil: 'domcontentloaded' };
  • In test/E2ETesting/imageLoading.spec.ts, updated the page.goto call to use { waitUntil: 'domcontentloaded' };
  • In test/E2ETesting/languageSelect.spec.ts, updated the page.goto call to use { waitUntil: 'domcontentloaded' };
  • In test/E2ETesting/nav.spec.ts, updated the page.goto call to use { waitUntil: 'domcontentloaded' };
  • In test/E2ETesting/pageLoading.spec.ts, updated the page.goto call to use { waitUntil: 'domcontentloaded' };
  • In test/E2ETesting/themeToggle.spec.ts, updated the page.goto call to use { waitUntil: 'domcontentloaded' };
  • In test/E2ETesting/umami.spec.ts, updated the page.goto calls to use { waitUntil: 'load' };

• Added a new fixture file (test/E2ETesting/fixtures.ts) to block all image requests from external domains;

• Updated various test files to use this new fixture, ensuring that network requests are controlled during tests;

Notes

The new fixture provides better isolation for tests, reducing flakiness due to external CDN requests.
The Lightbox navigates to next image test in gallery.spec.ts still is still CDN dependent, because the test compares the current image to the next image to check if it actually navigated to the next image. Without it actually loading the images, it would be comparing null to null, so, until I can mock this, it will stay network dependent.