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 user defined mutation filter #1341

Merged
merged 6 commits into from
Jun 27, 2022
Merged

Add user defined mutation filter #1341

merged 6 commits into from
Jun 27, 2022

Conversation

mbj
Copy link
Owner

@mbj mbj commented Jun 26, 2022

No description provided.

@mbj mbj mentioned this pull request Jun 26, 2022
@mbj mbj force-pushed the add/mutation-filter branch 6 times, most recently from 49e4049 to 04017b7 Compare June 26, 2022 03:27
@mbj mbj self-assigned this Jun 26, 2022
mbj added 6 commits June 26, 2022 23:42
* While this removes some mutatiosn inside the captures there is no test
  that pulls them into existance.
* Hence its better to remove them for the moment instead to rely on this
  logic.
* This is easier to extend.
Mutant will not emit mutations for AST patterns defined by the new
mutation specific option `ignore_pattern`.

```ruby

def foo
  logger.log('Before A')
  do_a
  logger.log('After A')
end
```

This will not emit any mutatiosn for both logger statements.

The granularity of the ignore pattern can be more or less specific, this
example matched all send nodes with the selector `log`. But it could
have requied the receiver `logger` via:

```
send
  { selector = log
  , receiver = send
    { selector = logger
    }
```
@mbj mbj force-pushed the add/mutation-filter branch 2 times, most recently from 3e70c36 to c0d5200 Compare June 27, 2022 00:05
@mbj mbj merged commit 3b91402 into main Jun 27, 2022
@mbj mbj deleted the add/mutation-filter branch June 27, 2022 00:29
```
send
{ selector = info
, recevier = send{selector=logger}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: Spelling: recevier -> receiver

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants