Skip to content

fix: re-enable test coverage#1195

Merged
achingbrain merged 6 commits intomasterfrom
fix/re-enable-test-coverage
Oct 24, 2023
Merged

fix: re-enable test coverage#1195
achingbrain merged 6 commits intomasterfrom
fix/re-enable-test-coverage

Conversation

@achingbrain
Copy link
Copy Markdown
Member

@achingbrain achingbrain commented Feb 20, 2023

c8 has been released recently as has node so re-enable test coverage

@achingbrain
Copy link
Copy Markdown
Member Author

achingbrain commented Feb 21, 2023

It's still broken. The CI jobs with coverage hang after completion then are killed after six hours.

@achingbrain achingbrain marked this pull request as draft February 21, 2023 07:04
@achingbrain
Copy link
Copy Markdown
Member Author

achingbrain commented Feb 21, 2023

Refs: bcoe/c8#454

@laurentsenta laurentsenta mentioned this pull request Mar 7, 2023
Open
@achingbrain achingbrain mentioned this pull request Apr 4, 2023
Closed
@whizzzkid
Copy link
Copy Markdown
Contributor

whizzzkid commented Apr 4, 2023

@achingbrain can we get this merged? what needs to happen in order to validate this? I tested this on m1-mac and seems to work well.

You also might wanna remove the --cov arg from the command.

@whizzzkid whizzzkid mentioned this pull request Apr 4, 2023
Open
24 tasks
@achingbrain
Copy link
Copy Markdown
Member Author

achingbrain commented Apr 4, 2023

can we get this merged?

Unfortunately coverage is still broken so not yet. See how CI on this PR is hanging on node jobs with coverage enabled.

You also might wanna remove the --cov arg from the command.

No, it's fine as it is. The workaround at the moment is to detect usage of coverage and print a warning to the user which explains why they aren't getting coverage.

Merging this PR will undo that and coverage will flow again, but it's blocked on bcoe/c8#454

Please do investigate alternatives in the meantime though.

@whizzzkid
Copy link
Copy Markdown
Contributor

whizzzkid commented Apr 4, 2023

@achingbrain got it, reading through the reported bug I don't think anyone knows what's causing it. It's hard to repro on local, GA being GA, I cancelled the workflow as it just hung.

@achingbrain achingbrain mentioned this pull request May 23, 2023
Merged
@achingbrain achingbrain force-pushed the fix/re-enable-test-coverage branch from f10ec13 to 78ea214 Compare June 10, 2023 06:17
@achingbrain
fix: re-enable test coverage
79b384a 
c8 has been released recently as has node so re-enable test coverage
@achingbrain
test: test on lts and current
cbe463b
@achingbrain achingbrain force-pushed the fix/re-enable-test-coverage branch from 78ea214 to cbe463b Compare June 10, 2023 07:26
achingbrain
achingbrain commented Oct 24, 2023
View reviewed changes
@achingbrain achingbrain marked this pull request as ready for review October 24, 2023 06:09
@achingbrain
Copy link
Copy Markdown
Member Author

achingbrain commented Oct 24, 2023

Merging as this appears to be working now.

@achingbrain achingbrain merged commit 9b35013 into master Oct 24, 2023
@achingbrain achingbrain deleted the fix/re-enable-test-coverage branch October 24, 2023 06:09
github-actions bot pushed a commit that referenced this pull request Oct 24, 2023
@semantic-release-bot
chore(release): 41.0.6 [skip ci]
90bf8a9 
## [41.0.6](v41.0.5...v41.0.6) (2023-10-24)

### Bug Fixes

* re-enable test coverage ([#1195](#1195)) ([9b35013](9b35013))
@achingbrain achingbrain restored the fix/re-enable-test-coverage branch October 25, 2023 06:23
achingbrain added a commit that referenced this pull request Oct 25, 2023
@achingbrain
Revert "fix: re-enable test coverage (#1195)"
7f5ab39 
This reverts commit 9b35013.
@achingbrain achingbrain mentioned this pull request Oct 25, 2023
Merged
achingbrain added a commit that referenced this pull request Oct 25, 2023
@achingbrain
fix: disable test coverage again (#1390)
cfeffc2 
This is not fixed, it's still causing larger builds to hang.

Reverts #1195
github-actions bot pushed a commit that referenced this pull request Oct 25, 2023
@semantic-release-bot
chore(release): 41.0.7 [skip ci]
6c36cfd 
## [41.0.7](v41.0.6...v41.0.7) (2023-10-25)

### Bug Fixes

* disable test coverage again ([#1390](#1390)) ([cfeffc2](cfeffc2)), closes [#1195](#1195)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@achingbrain @whizzzkid