1.1.1: pytest is failing in `tests/tests.py::TestFunctions::test_is_file` unit about polib

[rodrigoA]

Hi @kloczek,

Looks like another UT framework has been used. Here the unittest is used instead of pytest.

When, at release 1.1.1, I run them like python tests/tests.py or even runtests.sh, I get no errors.

I don't know why only this one fails with pytest.

[kloczek]

Correctly written uniitest test suite can be used OOTB with pytest.
What is the sense to perform assertion out of fixed value?

You will never get new warnings or errors because no one is working on unittest.
At ehe moment+8/9 of all modules is using pytest.
Please try to have look on https://docs.pytest.org/en/7.0.x/reference/plugin_list.html. Maybe this will give you a clue why so many python modules are using pytest as base test suite framework ..

[kthy]

Correctly riyyen uniitest test suite can be used OOTB with pytest.
What is the sense to perform assertion out of fixed value?

Ironically, I assume you mean "correctly written"?

The assertion does not have a fixed value. It asserts that calling polib._is_file(True) returns False. This test guards against polib._is_file('/some/path/to.po') returning True simply because the string is truthy.

I prefer Pytest myself, but if you come barging into a FOSS project, accuse the authors of using a library incorrectly and then try to bully them into fixing your problem simply by appeal to authority, don't be surprised if the response is less than enthusiastic.

[kloczek]

I prefer Pytest myself, but if you come barging into a FOSS project, accuse the authors of using a library incorrectly and then try to bully them into fixing your problem simply by appeal to authority, don't be surprised if the response is less than enthusiastic.

My comment was about the CODE .. not you.