-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[ci] Handle coverage missing on tests omitted by the retry. #14755
Conversation
Changelog[uncommitted] (2023-12-07) |
8350d93
to
3cf6df3
Compare
pkg/testing/integration/program.go
Outdated
requirementsPath := filepath.Join(cwd, "requirements.txt") | ||
requirementsmd5, err := hashFile(requirementsPath) | ||
if err != nil { | ||
return err | ||
} | ||
pt.opts.virtualEnvDir = fmt.Sprintf("pulumi-venv-%x", requirementsmd5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that there's some sort of interaction causing tests to fail in setup when different tests are reading and writing to the same virtualenv.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5146153
to
5449125
Compare
gotestyourself/gotestsum#274 Updates gotestsum to v1.11.0
8f28a68
to
ec921c9
Compare
ec921c9
to
351a77e
Compare
Workaround appears not to work on our setup. |
Fixes #14756
Due to tests being flakey, they are re-run. The re-runs cause coverage on well-behaved tests to be discarded for the re-run test coverage.
This PR aims to make tests and coverage more reliable.