p53209761
/
test_ok2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

minor: visit_Assert: move setting of `negation` out of branches

This commit is contained in:
Daniel Hahler 2019-11-18 20:26:31 +01:00
parent 64d8910516
commit f38f2d402e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/_pytest/assertion/rewrite.py
View File

@ -807,8 +807,9 @@ class AssertionRewriter(ast.NodeVisitor):
) )
) )
negation = ast.UnaryOp(ast.Not(), top_condition)
if self.enable_assertion_pass_hook: # Experimental pytest_assertion_pass hook if self.enable_assertion_pass_hook: # Experimental pytest_assertion_pass hook
negation = ast.UnaryOp(ast.Not(), top_condition)
msg = self.pop_format_context(ast.Str(explanation)) msg = self.pop_format_context(ast.Str(explanation))
# Failed # Failed
@ -860,7 +861,6 @@ class AssertionRewriter(ast.NodeVisitor):
else: # Original assertion rewriting else: # Original assertion rewriting
# Create failure message. # Create failure message.
body = self.expl_stmts body = self.expl_stmts
negation = ast.UnaryOp(ast.Not(), top_condition)
self.statements.append(ast.If(negation, body, [])) self.statements.append(ast.If(negation, body, []))
if assert_.msg: if assert_.msg:
assertmsg = self.helper("_format_assertmsg", assert_.msg) assertmsg = self.helper("_format_assertmsg", assert_.msg)