Skip to content

Feature/e2e tests poc#187

Merged
warm-coolguy merged 23 commits into
mainfrom
feature/e2e-tests-poc
Sep 30, 2024
Merged

Feature/e2e tests poc#187
warm-coolguy merged 23 commits into
mainfrom
feature/e2e-tests-poc

Conversation

@warm-coolguy
Copy link
Copy Markdown
Member

@warm-coolguy warm-coolguy commented Sep 19, 2024

Summary

Added playwright as E2E framework with rather trivial example test. With this, we can test some "big picture" effects.

Instructions for local reproduction and review

npm i, npx playwright install to install browsers for your system once (from what I understand, this has to be done once per executing machine), npm run test:e2e.

TODO

  • Add test for draw tool (actually drawing in map to have an example for canvas interactions)

@warm-coolguy warm-coolguy added the enhancement New feature or request label Sep 19, 2024
@warm-coolguy warm-coolguy self-assigned this Sep 19, 2024
@dopenguin
Copy link
Copy Markdown
Member

The pipeline for the E2E tests are currently failing. 🏓 me, once the lights are green!

@warm-coolguy
Copy link
Copy Markdown
Member Author

The pipeline for the E2E tests are currently failing. 🏓 me, once the lights are green!

🏓 @dopenguin

Also added another test to display canvas interaction possibilities.

Copy link
Copy Markdown
Member

@dopenguin dopenguin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread .gitignore Outdated
Comment thread playwright.config.ts
remove redundancy
Remove unnecessary comments
@warm-coolguy
Copy link
Copy Markdown
Member Author

🏓 @warm-coolguy

🏓 @dopenguin

Copy link
Copy Markdown
Member

@dopenguin dopenguin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@warm-coolguy warm-coolguy merged commit 0e54caa into main Sep 30, 2024
@warm-coolguy warm-coolguy deleted the feature/e2e-tests-poc branch September 30, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants