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

Add an assertion to the tests that should be loaded the target. #25958

Merged
merged 1 commit into from
Jul 27, 2016

Conversation

kamipo
Copy link
Member

@kamipo kamipo commented Jul 26, 2016

These test cases tests exactly mutating loaded target.

These test cases tests exactly mutating loaded target.
@rails-bot
Copy link

r? @senny

(@rails-bot has picked a reviewer for you, use r? to override)

@senny
Copy link
Member

senny commented Jul 26, 2016

@kamipo thanks, let's wait for Travis to pass.

@senny senny merged commit 7e6996a into rails:master Jul 27, 2016
@senny
Copy link
Member

senny commented Jul 27, 2016

@kamipo thanks 💛

@kamipo kamipo deleted the should_be_target_loaded branch July 27, 2016 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants