-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Allow to reset async for ExUnit.Case by using it again #9360
Allow to reset async for ExUnit.Case by using it again #9360
Conversation
test "async must be true", context do | ||
assert context.async | ||
end | ||
end |
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.
Couldn't find a better way to check the value saved to async
attribute.
So I'm confused: the original issue #9355 is about warning if |
We want to allow it to be called multiples times because of case template. So we are fixing it so it works as expected if called multiple times (i.e. latest option wins). |
@josevalim got it! Thanks @kelvinst! 💟 |
Closes #9355