Skip to content

Add test coverage support for karma and playwright test suites - #1252

Draft
juliandescottes wants to merge 5 commits into
piskelapp:masterfrom
juliandescottes:test-coverage
Draft

Add test coverage support for karma and playwright test suites#1252
juliandescottes wants to merge 5 commits into
piskelapp:masterfrom
juliandescottes:test-coverage

Conversation

@juliandescottes

@juliandescottes juliandescottes commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator
  • Adds configuration for karma and playwright test coverage.
  • Adds two new build targets: npm run e2e:coverage and npm run unit-test:coverage.
  • Adds codecov setup and github actions for coverage.
  • Fixes tests which failed on Chrome update due to different image compression algorithm (also would be flaky on different hardware anyway)

@juliandescottes
juliandescottes force-pushed the test-coverage branch 5 times, most recently from f1d72a8 to 975b761 Compare April 6, 2026 21:39
@juliandescottes
juliandescottes marked this pull request as ready for review April 6, 2026 21:41
@codecov

codecov Bot commented Apr 6, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@juliandescottes
juliandescottes force-pushed the test-coverage branch 2 times, most recently from 05ed655 to 8a4486c Compare April 8, 2026 20:47
@juliandescottes

Copy link
Copy Markdown
Collaborator Author

After discussing, while we're happy to have an initial coverage data point, it's not necessarily something we want to maintain in the long run. Let's keep the PR on the side to potentially rebase and re-run it after some additional e2e have been added.

@juliandescottes
juliandescottes marked this pull request as draft April 9, 2026 20:42
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.

1 participant