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 parent array for coverage #655
Use parent array for coverage #655
Conversation
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.
LGTM
could you please rebase on master? |
a7df6f3
to
9693953
Compare
Appveyor failure is due to a http request failing for schema validation. |
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.
👍 Thank you @BackEndTea
Thank you @borNfreee |
@borNfreee Is there a new release with this bugfix planned soon? |
This PR:
As shown here, array coverage is a bit wonky, and not all items are considered covered, except if function/methods are called as part of the array.
This PR should make sure that any array is properly covered.
It isn't properly tested, as the MutationsCollectorVisitor is a bit complex. I'll create a separate PR, to fix that up later.
Fixes #652