Is there a way to let Pants treat pytest's exit code 5 as a successful run?

Shitty solution: a pytest “plugin” that just provides an empty test if none have been collected

You might try something like https://pypi.org/project/pytest-custom-exit-code/

I'm still open to the idea of changing Pants to tolerate it though. We've gone back and forth on what to do and tended to the conservative approach of mirroring the underlying tool The argument for keeping it is that it's a useful warning when you forgot to implement test But then it gets annoying in other cases, especially

-m

and

-k

like you say We could add an option, but what should the default be? And ugh options bloat

I think an option to coerce this to 0 makes sense

with default to what?

I'm personally conflicted here 😞

how so?

We're coercing the behavior of the underlying tool. Which that error exists for a reason (although I don't completely agree). Idealistically Pants sits on top of the tools and doesn't have much in between. The less we change, the more users can intuit. OTOH our execution strategy is not what the tools are built to expect, and having users fall into failure isn't good.

We're coercing the behavior of the underlying tool. Which that error exists for a reason (although I don't completely agree). Idealistically Pants sits on top of the tools and doesn't have much in between. The less we change, the more users can intuit.

Yeah, that's my general philosophy. Pants v1 did much more customization of tools, e.g. having a custom implementation of Flake8 builtin lol

Instead we could nudge people towards the pytest-way of solving this and treat this as an opportunity to educate on how Pants operates under-the-hood

I suspect it happens far more frequently w/ Pants than non-Pants repos because we run test files one-process-per-file

Yes, I think this is the main reason why the Pants team is facing this question...

Oh, does pytest have the option to turn this off?

pytest itself does not, but we can install

custom_exit_code

plugin

We can document that as a workaround