Move a comment to work around a pytest bug.
Review Request #2201 - Created May 11, 2015 and submitted
When that test fails, pytest attempts to construct a verbose backtrace.
However that comment in the middle of a multiline list literal tickles
a bug in the backtrace construction code that causes pytest to fail hard
with an INTERNALERROR, instead of showing the details of the underlying bug.
While debugging this, I noticed that we don't constrain the versions of
pytest that we bake into the test chroot, so it will end up being a
different version than the one in 3rdparty/python/requirements.txt.
This is pretty confusing, so I added the relevant constraints.
See here for an example of this useless INTERNALERROR dump:
Relevant test passes locally in otherwise unmodified repo.
Relevant test fails with INTERNALERROR when test is deliberately broken before this change.
Relevant test fails with useful backtrace when deliberately broken after this change.
CI pending: https://travis-ci.org/pantsbuild/pants/builds/62150849
An install need not have 3rdparty requirements for any of these in which case its not so confusing. I'm in favor of pinning what we support though for now. I think the right thing TODO is as you suggest - ie: add config options for overriding these specs so users need not wait on a pants release to use that new pytest feature they want.
Revision 2 (+6 -3)
Status: Closed (submitted)
Submitted as d20a43faebe7b9c173edefd3940bdf96342f1fb4.