Notes and clarifications for the reviewer

Mapping-related tests were failing rather often with AssertionError.

Why? The object tested strings but a null was given (the issue was identified after temporarily adding logging).

The cause: race conditions in partial page loads where DOM elements were accessed before fully loaded.

Changes Made

concept-full-vs-partial.cy.js

• Notation clipboard test
  Lines 79-81
  Ensures partial page load completes before copying notation. Previously copied wrong value (33.10 instead of 33.2).

• Mapping stabilization wait
  Lines 150, 153-155
  Prevents race condition where mappings reload after initial spinner check.

• Explicit mapping label wait
  Lines 160-161
  Ensures mapping labels exist before accessing their content.

• Changed .contains() to .invoke('text').should('contain')
  Lines 158, 164, 166, 171 an 173
  More explicit assertions, consistent pattern and hopefylly easier debugging.

• Hash namespace test
  Lines 192-193 sand 199-209
  The same pattern as labyrinths test - explicit label wait with 20 s timeout and explicit text checking.

Summa summarum:

• All 4 unstable mapping tests now pass consistently
• Notation clipboard test fixed (was expecting 33.2, got 33.10)
• Minimal changes, only essential stabilization added
• No functionality changes (only timing/reliability improvements)

concept.cy.js

• Property override test - explicit text check and comment
  Lines 139-141 (added line 140, modified 141)
  The same more explicit text checking pattern. Added timeout lining up with other similiar parts. Improved commenting.

Summa Summarum:

Replaces .contains('text') with .invoke('text').should('contain', 'text') for explicit text checking and adds stabilization waits after spinner checks. Adds explicit waits for mapping labels (20 s timeout) and adds heading verification for partial page loads before accessing content.

• All 4 unstable mapping tests now pass consistently
• Notation clipboard test fixed (was expecting 33.2, got 33.10)
• Property override test made consistent with the same pattern
• Essential stabilization and consistency improvements

Post Scriptum:
.invoke('text') always asserts on the resolved final string, which makes it a better choice in tests where the expected output is not 100 percent certain. It is also less sensitive to DOM timing quirks and makes debugging easier.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.32%. Comparing base (a5202dd) to head (33c09e1).
⚠️ Report is 2 commits behind head on main.

@@            Coverage Diff            @@
##               main    #1906   +/-   ##
=========================================
  Coverage     70.32%   70.32%           
  Complexity     1658     1658           
=========================================
  Files            34       34           
  Lines          4364     4364           
=========================================
  Hits           3069     3069           
  Misses         1295     1295           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud