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.

Sign up for GitHub

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

Jump to bottom

Generate coverage value for both unit and E2E tests #6501

Closed
cnotv opened this issue Jul 26, 2022 · 1 comment · Fixed by #6690
Closed

Generate coverage value for both unit and E2E tests #6501

cnotv opened this issue Jul 26, 2022 · 1 comment · Fixed by #6690
Assignees
@cnotv
Labels
area/test Test (e2e and unit) kind/tech-debt Technical debt size/8 Size Estimate 8 status/dev-validate
Milestone
v2.7.0

Comments

@cnotv
Copy link
Contributor

cnotv commented Jul 26, 2022 •
edited
Loading

Detailed Description
Compute and display in a dashboard the merged coverage values and analysis for both Jest and Cypress tests.

Context
Several operations are required:

  • Cleanup coverage report from unit test
  • Generate coverage from E2E tests
  • Merge coverage values and lines for the 2 tests
  • Register Codecov to collect data on complete
  • Review GH Actions so the metrics are not polluted
    • Add push to master
    • Review id name for push cases
    • Prefix names with ID values and add run_attempt

Example report Cypress: https://docs.cypress.io/guides/tooling/code-coverage#Full-stack-code-coverage
@cnotv cnotv added area/test Test (e2e and unit) kind/tech-debt Technical debt area/unit-test Issues involving need, configuration or changes of unit test labels Jul 26, 2022
@cnotv cnotv added this to the v2.7.0 milestone Jul 26, 2022
@cnotv cnotv self-assigned this Jul 26, 2022
@nwmac nwmac removed the area/unit-test Issues involving need, configuration or changes of unit test label Jul 28, 2022
@zube zube bot removed the [zube]: Next Up label Aug 10, 2022
@cnotv cnotv added area/test Test (e2e and unit) size/8 Size Estimate 8 and removed area/test Test (e2e and unit) labels Aug 10, 2022
@cnotv
Copy link
Contributor Author

cnotv commented Aug 18, 2022 •
edited
Loading

Looks like the folder defined in the path of the artifact action is now omitted actions/upload-artifact#30

@cnotv cnotv mentioned this issue Aug 18, 2022
Merged
@nwmac nwmac modified the milestones: v2.7.0, v2.6.9 Aug 30, 2022
@gaktive gaktive modified the milestones: v2.6.9, v2.7.0 Sep 16, 2022
@zube zube bot removed the [zube]: Done label Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cnotv cnotv

Labels
area/test Test (e2e and unit) kind/tech-debt Technical debt size/8 Size Estimate 8 status/dev-validate
Projects
None yet
Milestone
v2.7.0
Development

Successfully merging a pull request may close this issue.

Generate coverage value for both unit and E2E tests cnotv/dashboard
4 participants
@nwmac @cnotv @gaktive @richard-cox