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

Wrap Rspec before and example blocks with a mutex for JRuby #537

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

bensheldon
Copy link
Owner

@bensheldon bensheldon commented Mar 3, 2022

A better implementation of 205 / fc548e7#diff-053aa3ea46cb7c6a649e0d0fc592ce729b66e1d44be475516fd58857ba695862R14 which in hindsight doesn't actually look functional because it wraps the entire before/example/after with a mutex, rather than wrapping individual components.

The intention is to avoid this sort of error when a class method is stubbed:

Screen Shot 2022-03-02 at 7 21 17 PM

@bensheldon bensheldon temporarily deployed to goodjob-jruby-mutex-kixuee3f6c March 3, 2022 03:06 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-jruby-mutex-kixuee3f6c March 3, 2022 03:11 Inactive
@bensheldon bensheldon merged commit 4737d40 into main Mar 3, 2022
@bensheldon bensheldon deleted the jruby_mutex branch March 3, 2022 03:56
@bensheldon bensheldon added the refactor Code changes that do not introduce new features label Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code changes that do not introduce new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant