UpSetPlot icon indicating copy to clipboard operation
UpSetPlot copied to clipboard

Test suite issues when using pytest 8

Open tillea opened this issue 1 year ago • 2 comments

Hi, the Debian packaged version of UpSetPlot received two related bug reports about failing its build time test as well as its CI test (in Debian terminology autopkgtest) which occured since the switch to pytest 8.0.2 (formerly 7.4.4, where all test worked). You can see a full test log in our CI. While the problem was reported against UpSetPlot 0.8.0 I verified that the problem persists in 0.9.0.

To work around the problem I marked the only affected test xfail in a patch. I guess the issue is caused by this line of code which I kindly leave to your inspection. Please note that I have no idea about UpSetPlot itself, I'm not maintaining the package inside Debian just had some dependencies affected and tried this workaround.

Kind regards, Andreas.

tillea avatar Mar 18 '24 10:03 tillea

https://docs.pytest.org/en/latest/how-to/capture-warnings.html#additional-use-cases-of-warnings-in-tests

I think adding

    with warnings.catch_warnings():
        warnings.simplefilter("error")

in place of the with pytest.warns(None): should fix this

mr-c avatar Mar 18 '24 11:03 mr-c

Good hint, uploaded Debian package with this change, Andreas.

tillea avatar Mar 18 '24 12:03 tillea