floraachy
/
pytest2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add comment about why we remove docstrings on test_assertrewrite 

As explained in pytest-dev/pytest#2870
This commit is contained in:
Bruno Oliveira 2017-10-26 20:15:05 -02:00 committed by GitHub
parent fd7bfa30d0
commit 27bb2eceb4
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
testing/test_assertrewrite.py
View File

@ -65,6 +65,9 @@ class TestAssertionRewrite(object):
def test_place_initial_imports(self): def test_place_initial_imports(self):
s = """'Doc string'\nother = stuff""" s = """'Doc string'\nother = stuff"""
m = rewrite(s) m = rewrite(s)
# Module docstrings in 3.7 are part of Module node, it's not in the body
# so we remove it so the following body items have the same indexes on
# all Python versions
if sys.version_info < (3, 7): if sys.version_info < (3, 7):
assert isinstance(m.body[0], ast.Expr) assert isinstance(m.body[0], ast.Expr)
assert isinstance(m.body[0].value, ast.Str) assert isinstance(m.body[0].value, ast.Str)