Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate code coverage in container #4105

Merged
merged 28 commits into from
Mar 6, 2025

Conversation

walmazacn
Copy link
Contributor

@walmazacn walmazacn commented Jan 8, 2025

Description

Changes proposed in this pull request:

  • adds E2E code coverage stats in container
  • adds script to unify coverage for both E2E and unit tests

Related issue(s)

Resolves #4151

@walmazacn walmazacn linked an issue Jan 8, 2025 that may be closed by this pull request
@walmazacn
Copy link
Contributor Author

It seems it's possible to have both E2E and unit tests covered in one report, but only if they rely on Cypress:
https://docs.cypress.io/app/tooling/code-coverage#E2E-and-unit-code-coverage

@walmazacn walmazacn marked this pull request as ready for review January 29, 2025 07:26
@walmazacn walmazacn force-pushed the 4080-investigate-code-coverage-for-cypress branch from 809553f to a85c703 Compare February 6, 2025 13:19
@walmazacn walmazacn force-pushed the 4080-investigate-code-coverage-for-cypress branch from 3fb14d1 to d79384f Compare February 11, 2025 13:52
@JohannesDoberer JohannesDoberer self-assigned this Feb 17, 2025
@walmazacn walmazacn changed the title Investigate code coverage for cypress Investigate code coverage in container Feb 18, 2025
@walmazacn walmazacn linked an issue Feb 18, 2025 that may be closed by this pull request
Copy link
Contributor

@JohannesDoberer JohannesDoberer left a comment

Choose a reason for hiding this comment

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

LGTM

@walmazacn walmazacn merged commit 1681a49 into main Mar 6, 2025
11 checks passed
@walmazacn walmazacn deleted the 4080-investigate-code-coverage-for-cypress branch March 6, 2025 12:41
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.

Add test coverage in container
3 participants