-
-
Notifications
You must be signed in to change notification settings - Fork 152
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 inline disable configuration #1320
Conversation
9e60f0b
to
8e350d7
Compare
|
||
let(:expected_associations) do | ||
associations = {} | ||
associations.compare_by_identity |
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.
This is forced by parsers comment associator, I'm personally not super in favor to use object memory identity in any kind of API.
I'd prefer parser would use node index to associate, but I cannot change its API.
end | ||
|
||
it 'returns parsed source' do | ||
expect(subject).to eql(s(:sym, :source)) | ||
expect(subject.inspect).to eql( |
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.
Comparing via inspect as Parser::Source::*
only does identity on #==
and #eql?
. And writing peeking assertions is not as exhaustive. Basically an in-memory golden test :P
fff7aa5
to
9e9f409
Compare
* This allows to place `mutant:disable` comments next to subjects to mark them as disabled. * Inheritance is not supported currently, so a comment on a class will not make this entire class disabled.
d52ff56
to
38f1e1c
Compare
No description provided.