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

use forking-test-runner to ensure no test pollution between different… #593

Merged
merged 1 commit into from
Jan 3, 2023

Conversation

grosser
Copy link
Contributor

@grosser grosser commented Dec 22, 2022

… test files

maybe this cuts down on the random errors
... and even if not it should reduce wild goose chases if one test messes with global state

@grosser
Copy link
Contributor Author

grosser commented Dec 22, 2022

welp ... truffle-ruby does not have fork 😞

@grosser
Copy link
Contributor Author

grosser commented Dec 22, 2022

fixed that ... still errors seems to be 98 and 135 mostly :/

@grosser grosser force-pushed the grosser/fork branch 7 times, most recently from 355adae to 193179b Compare December 22, 2022 05:28
@grosser
Copy link
Contributor Author

grosser commented Dec 22, 2022

fixed the errors in #594 ... this PR needs a rebase once that's merged

@cben cben closed this Jan 3, 2023
@cben cben reopened this Jan 3, 2023
@cben
Copy link
Collaborator

cben commented Jan 3, 2023

All green now with #594. You still consider this desired, right?

Oh, it's your gem. Looks very nice! 👏

@cben cben merged commit 71c5a80 into ManageIQ:master Jan 3, 2023
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.

None yet

2 participants