-
Notifications
You must be signed in to change notification settings - Fork 62
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
Explicit targets not registered globally #364
Explicit targets not registered globally #364
Conversation
Ensure that targets registered for one test does not get used in another.
… registered targets
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.
Taking another look at this, and my initial doubts was in changing logic.Iterator
by adding the targets
argument. But I think that's the way to go. Before, it kept up to date with tagets by looking globally, but that was no good. Now it's being handed targets directly, which I think is right.
Have a look at my minor notes, and then I think we can merge this. Good job.
Thanks @buddly27! |
This resolves #363 and #345