Regression in 3.7.0: AttributeError for _collectfile

[fschulze]

I get AttributeError: 'FlakesItem' object has no attribute '_collectfile' when running devpi-server tests with tox (tox -e py27 is enough) since pytest 3.7.0. When I pin to pytest<3.7dev it works (using 3.6.4).

It seems to be related to the current working directory. The tox config switches to test_devpi_server. When I run pytest in server it's fine, when i run in server/test_devpi_server it breaks.

[The-Compiler]

Not sure if this is an issue in pytest itself, or the pytest-flakes plugin.

[pytestbot]

GitMate.io thinks possibly related issues are #707 (pytest-xdist regression since execnet 1.3.0), #2968 (Memory usage in 3.3.0), #1083 (Regression with parallel tests on 2.7.3 and 2.8), #386 (tests fail with pexpect-3.0), and #1572 (Making _CallOutcome.result private in 3.0).

[RonnyPfannschmidt]

this is a regression made by pytest - root._collectfile got introduced in #3389

this triggered a unintentional regression as now a private api is used in more places and on objects that dont belong to it

[JohanLorenzo]

Hello! Thank you for this great library! The Mozilla release engineering team relies on it for several projects 😃

I just found a similar regression:

AttributeError: 'async_generator' object has no attribute 'put'

This async generator is a regular aiohttp.ClientSession() and is initialized here. This might be an integration issue with the pytest-asyncio plugin.

I rolled pytest back to 3.6.4 and things are back working. You can see both the failure and the workaround in

• Fix fake_session breakage mozilla-releng/beetmoverscript#176
• https://travis-ci.org/mozilla-releng/beetmoverscript/jobs/410250756#L598

[RonnyPfannschmidt]

@JohanLorenzo this is a different issue - related to fixture definitions, please open a new issue

[JohanLorenzo]

Thanks for the quick answer. I opened #3747

[Thomasillo]

Same here with flake8:

$ py.test --flake8 package
=========================================================================================== test session starts ============================================================================================
platform darwin -- Python 3.6.2, pytest-3.7.0, py-1.5.3, pluggy-0.7.1
rootdir: /Users/Thomas/Work/stuff, inifile: setup.cfg
plugins: flake8-1.0.1, cov-2.5.1, hypothesis-3.57.0
collected 0 items / 1 errors                                                                                                                                                                               

================================================================================================== ERRORS ==================================================================================================
____________________________________________________________________________________________ ERROR collecting  _____________________________________________________________________________________________
../../../../.local/miniconda3/envs/test/lib/python3.6/site-packages/_pytest/runner.py:201: in __init__
    self.result = func()
../../../../.local/miniconda3/envs/test/lib/python3.6/site-packages/_pytest/runner.py:261: in <lambda>
    call = CallInfo(lambda: list(collector.collect()), "collect")
../../../../.local/miniconda3/envs/test/lib/python3.6/site-packages/_pytest/main.py:475: in collect
    for x in self._collect(arg):
../../../../.local/miniconda3/envs/test/lib/python3.6/site-packages/_pytest/main.py:519: in _collect
    for x in root._collectfile(pkginit):
E   AttributeError: 'Flake8Item' object has no attribute '_collectfile'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
========================================================================================= 1 error in 0.17 seconds ==========================================================================================

[ThreePinkApples]

Getting the same with PEP8

============================= test session starts ==============================
platform linux -- Python 3.6.4, pytest-3.7.0, py-1.5.4, pluggy-0.7.1
rootdir: /var/lib/jenkins/jobs/Simplereg/workspace, inifile:
plugins: xdist-1.22.5, twisted-1.8, pep8-1.0.6, forked-0.2, cov-2.5.1
collected 0 items / 1 errors

==================================== ERRORS ====================================
______________________________ ERROR collecting  _______________________________
.tox/py36/lib/python3.6/site-packages/_pytest/runner.py:201: in __init__
    self.result = func()
.tox/py36/lib/python3.6/site-packages/_pytest/runner.py:261: in <lambda>
    call = CallInfo(lambda: list(collector.collect()), "collect")
.tox/py36/lib/python3.6/site-packages/_pytest/main.py:475: in collect
    for x in self._collect(arg):
.tox/py36/lib/python3.6/site-packages/_pytest/main.py:519: in _collect
    for x in root._collectfile(pkginit):
E   AttributeError: 'Pep8Item' object has no attribute '_collectfile

[cdeil]

We're seeing this error after the update pytest 3.6 -> 3.7:

__________________________ ERROR collecting  _______________________________
�[1m�[31m/home/travis/miniconda/envs/test/lib/python2.7/site-packages/_pytest/runner.py�[0m:201: in __init__
�[1m    self.result = func()�[0m
�[1m�[31m/home/travis/miniconda/envs/test/lib/python2.7/site-packages/_pytest/runner.py�[0m:261: in <lambda>
�[1m    call = CallInfo(lambda: list(collector.collect()), "collect")�[0m
�[1m�[31m/home/travis/miniconda/envs/test/lib/python2.7/site-packages/_pytest/main.py�[0m:475: in collect
�[1m    for x in self._collect(arg):�[0m
�[1m�[31m/home/travis/miniconda/envs/test/lib/python2.7/site-packages/_pytest/main.py�[0m:519: in _collect
�[1m    for x in root._collectfile(pkginit):�[0m
�[1m�[31mE   AttributeError: 'Pair' object has no attribute '_collectfile'�[0m
!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!

https://travis-ci.org/gammapy/gammapy/jobs/410288389#L2253

Same issue or a different one?

[RonnyPfannschmidt]

same all issues that are triggered by _collectfile missing on 3rd party items are due to this one
its an entirely unintended issue