palcarazm / bs-darkmode-toggle / 23998134099

98%

v1: 98%

push

github

refactor: restructure test app and Cypress E2E tests for better maintainability

What has been done:
- Consolidate duplicate HTML files into single test-app.html with dynamic
  script loader based on URL parameter
- Replace legacy test architecture with modular structure under test/app,
  test/common, test/loaders, and test/runners directories
- Remove deprecated BsDarkmodeToggleModel and TestAppModel from support
- Create new Cypress page objects (TestAppPage, DarkmodeToggleComponent)
- Add Interfaces helper to reduce code duplication across test files
- Reorganize Cypress E2E tests into initialization/ and methods/ directories
- Simplify test logic by extracting common checks into reusable module
- Fix checkRoot implementation using proper Cypress promise chaining
- Standardize code formatting with tabWidth 4 across all files

Why has been done:
- Previous architecture had tight coupling between test app and Cypress
  tests, making changes difficult and error-prone
- Duplicate HTML files for ECMAS and jQuery interfaces violated DRY principle
- Test runner logic was duplicated between test app and Cypress support files
- Cypress tests contained repetitive code blocks for each interface and check
- Asynchronous handling in checkRoot was causing timeout issues due to
  improper Cypress command chaining
- Refactoring improves separation of concerns, reduces code duplication
  by ~60%, and makes adding new test cases significantly easier
- Page Object pattern provides cleaner, more maintainable test code

131 of 135 branches covered (97.04%)

Branch coverage included in aggregate %.

172 of 172 relevant lines covered (100.0%)

90.0 hits per line