Skip to content

Parametrise Integration Tests#295

Open
Waguramu wants to merge 22 commits intomainfrom
parametrise-integration
Open

Parametrise Integration Tests#295
Waguramu wants to merge 22 commits intomainfrom
parametrise-integration

Conversation

@Waguramu
Copy link
Copy Markdown
Contributor

@Waguramu Waguramu commented Dec 18, 2025

TODOs:

  • Make sure the tests run faster / do not idle requesting the tiles.
  • Combine snapshot tests together where possible.
  • Enable the second view to initialise with the first view's parameters when opened.

@Waguramu Waguramu self-assigned this Dec 18, 2025
@Waguramu Waguramu marked this pull request as ready for review January 13, 2026 08:19
@github-actions
Copy link
Copy Markdown

Vitest coverage

  • Lines: 56.7% (4041/7126)
  • Statements: 56.7% (4041/7126)
  • Functions: 44.8% (270/603)
  • Branches: 71.5% (699/977)

Artifact: https://github.com/ndsev/erdblick/actions/runs/21030416467

@Waguramu
Copy link
Copy Markdown
Contributor Author

Waguramu commented Jan 16, 2026

  • Make sure to eliminate the issues with locating editor dialogue due to attach to body property.
  • Use dark mode
  • Use more area/full page for snapshots for visual appeal.
  • Use maps panel + map + docked inspections for the main snapshot.
  • Introduce a snapshot for the basic layout without the map panel.
  • Try to reproduce Source Data snapshots.
  • Add projection change, style change (and perspective change) to the multi-view snapshot.
  • Add two separate snapshots for style controls and style editor.
  • Possibly highlight selected locators.

@Waguramu Waguramu changed the base branch from main to release/2026.1.0 March 25, 2026 12:36
@Waguramu Waguramu force-pushed the parametrise-integration branch from 3641436 to ec69aef Compare March 26, 2026 08:09
Base automatically changed from release/2026.1.0 to main March 26, 2026 10:48
@Waguramu Waguramu force-pushed the parametrise-integration branch from ec69aef to d0158a2 Compare March 31, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants