Skip to content

Conversation

@sommerfe
Copy link
Contributor

@sommerfe sommerfe commented Dec 9, 2025

@sommerfe sommerfe self-assigned this Dec 9, 2025
@github-actions github-actions bot added the bug label Dec 9, 2025
@cypress
Copy link

cypress bot commented Dec 9, 2025

web-mapviewer    Run #6227

Run Properties:  status check errored Errored #6227  •  git commit 1c3117cace: PB-2064: fix opacity on print layers
Project web-mapviewer
Branch Review fix-pb-2064-print
Run status status check errored Errored #6227
Run duration 27m 25s
Commit git commit 1c3117cace: PB-2064: fix opacity on print layers
Committer Felix Sommer
View all properties for this run ↗︎

Test results
Tests that failed  Failures 42
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 3
Tests that passed  Passing 153
View all changes introduced in this branch ↗︎

Tests for review

Failed  layers.cy.ts • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
An uncaught error was detected outside of a test Test Replay Screenshots
Failed  drawing.cy.ts • 11 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Drawing module tests > Drawing mode/tools > can create marker/icons and edit them Test Replay Screenshots
Drawing module tests > KML management > deletes the drawing when confirming the delete modal Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer in the layer list / URL params correctly Test Replay Screenshots
Drawing module tests > KML management > keeps the KML after a page reload, and creates a copy if it is then edited Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer correctly if it comes attached with an adminId at startup Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer correctly if it comes attached with an adminId at startup from a legacy URL Test Replay Screenshots
Drawing module tests > KML management > receives an empty KML and can use drawing mode Test Replay Screenshots
Drawing module tests > others > can export the drawing/profile in multiple formats Test Replay Screenshots
Drawing module tests > others > generates short links when sharing a drawing Test Replay Screenshots
Drawing module tests > others > shows a profile of a line/measure coming from service-alti data Test Replay Screenshots
The first 10 failed tests are shown, see all 11 tests in Cypress Cloud.
Failed  timeSlider.cy.ts • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
An uncaught error was detected outside of a test Test Replay Screenshots
Failed  featureSelection.cy.ts • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
An uncaught error was detected outside of a test Test Replay Screenshots
Failed  shareShortLink.cy.ts • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
An uncaught error was detected outside of a test Test Replay Screenshots

The first 5 failed specs are shown, see all 16 specs in Cypress Cloud.

@sommerfe sommerfe force-pushed the fix-pb-2064-print branch 4 times, most recently from 31a1e57 to 1c3117c Compare December 15, 2025 07:43
@sommerfe sommerfe changed the base branch from feat-PB-1383-pinia-store to fix-pb-2026-fix-import-tool-file December 15, 2025 07:44
@ismailsunni ismailsunni force-pushed the fix-pb-2026-fix-import-tool-file branch from 313b278 to 06578d8 Compare December 16, 2025 02:56
ismailsunni and others added 29 commits December 18, 2025 08:50
- Add validation checks before store assertions for non-CORS GPX import
- Handle menu re-renders after layer removal with stabilization waits
- Add conditional warning window handling
- Move profile intercept setup before interactions
When importing a KML file that is outside the extent, the parser was
showing "Invalid file" error instead of the correct "out of bounds" error.

The issue occurred because parseAll() threw the first rejected error from
any parser, which was typically InvalidFileContentError from parsers that
couldn't handle the file format. However, the KML parser correctly
identified the file as KML and threw a more specific OutOfBoundsError.

Fixed by prioritizing specific errors (OutOfBounds, Empty, UnknownProjection)
over generic InvalidFileContentError, ensuring users see the most relevant
error message.
When importing a KML file that is outside the extent, the parser was
showing "Invalid file" error instead of the correct "out of bounds" error.

The issue occurred because parseAll() threw the first rejected error from
any parser, which was typically InvalidFileContentError from parsers that
couldn't handle the file format. However, the KML parser correctly
identified the file as KML and threw a more specific OutOfBoundsError.

Fixed by prioritizing specific errors (OutOfBounds, Empty, UnknownProjection)
over generic InvalidFileContentError, ensuring users see the most relevant
error message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants