-
Notifications
You must be signed in to change notification settings - Fork 10
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
Codemod: remove-assertion-in-pytest-raises #219
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #219 +/- ##
==========================================
- Coverage 96.45% 96.44% -0.01%
==========================================
Files 96 97 +1
Lines 4090 4169 +79
==========================================
+ Hits 3945 4021 +76
- Misses 145 148 +3
|
if not self.filter_by_path_includes_or_excludes( | ||
self.node_position(original_node) | ||
): | ||
return updated_node |
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.
you added a test_exclude_line
but still it says not covered.... weird.
Well actually, given this is a leave_with node, it's likely the same issue as the other codemods with this todo # TODO: add filter by include or exclude that works for nodes
. So I think you can go ahead and remove the unit test and just add the todo line
Quality Gate failedFailed conditions D Reliability Rating on New Code (required ≥ A) See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Overview
Adds a codemod that removes assertions from the bottom of
pytest.raises
context manager.