Add test coverage for tests/helpers.py

[abn]

As requested by @Secrus in https://github.com/python-poetry/poetry/pull/9149#discussion_r1525588514.

[dimbleby]

I don't understand. If there is code in tests/helpers.py that is not executed, just remove it already?

[DarkTechPirate]

No what are even supposed to do ?

[abn]

@dimbleby I think @Secrus is asking for test helpers to have functional test coverage. But yes, if we are not using a helper we should remove it. Although, I do not think that is the case right now.

[dimbleby]

I dont buy it: who writes tests for their tests? what then, shall we write tests for the tests for the tests too?

test code automatically gets coverage by being executed in the test suite, else delete it

[abn]

I'll let @Secrus decide what they want to do with this one as I just created the issue based on his request. :wink:

[dimbleby]

@Julynx you have completely misunderstood what is being discussed here

[Julynx]

@Julynx you have completely misunderstood what is being discussed here

Fair, I apologize for the misunderstading and I'm deleting my previous reply. Thanks for your patience.