Skip to content

test(core): add some tests#826

Open
gluecksmelodie wants to merge 2 commits intoci/coverage-reporterfrom
feature/add-tests
Open

test(core): add some tests#826
gluecksmelodie wants to merge 2 commits intoci/coverage-reporterfrom
feature/add-tests

Conversation

@gluecksmelodie
Copy link
Collaborator

@gluecksmelodie gluecksmelodie commented Oct 29, 2025

  • Based on ci(core): report code coverage in CI #809
  • Added some (AI-generated) e2e tests for regression
    • Unfortunately, e2e tests are not included in the test coverage as Stencil's E2E tests run in a browser via Puppeteer, so they don't contribute to Jest coverage.
  • Added the flag filter-changed-files: true to the coverage report so that coverage comments in a PR now only list the changed files, not all. The percentage still refers to the overall coverage, though.

@gluecksmelodie gluecksmelodie changed the base branch from main to ci/coverage-reporter October 29, 2025 15:24
@gluecksmelodie gluecksmelodie marked this pull request as draft October 29, 2025 15:24
@github-actions
Copy link
Contributor

Core Coverage Report

Coverage after merging feature/add-tests into main will be
34.95%
Coverage Report for Changed Files
FileStmtsBranchesFuncsLinesUncovered Lines

@github-actions
Copy link
Contributor

Angular Coverage Report

Coverage after merging feature/add-tests into main will be
24.23%
Coverage Report for Changed Files
FileStmtsBranchesFuncsLinesUncovered Lines

@gluecksmelodie gluecksmelodie marked this pull request as ready for review November 7, 2025 14:18
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