New review comment by tornaria on void-packages repository https://github.com/void-linux/void-packages/pull/36569#discussion_r846148315 Comment: That is ugly. Also, `-k` doesn't seem to support syntax like `-k not(TestWebPDFExporter::test_webpdf_without_chromium)`. One can get away with `--deselect` but this requires passing the complete prefix (which includes the full path to the python file where the test is located). I came up with this ``` make_check_args="-m not(network) -k not(test_webpdf_without_chromium)" ``` matching precisely the three tests that have to be disabled... but it's hard to extend, I'm almost inclined to prefer the completely explicit (but very verbose) ``` make_check_args=" --deselect nbconvert/exporters/tests/test_webpdf.py::TestWebPDFExporter::test_export --deselect nbconvert/exporters/tests/test_webpdf.py::TestWebPDFExporter::test_webpdf_without_chromium --deselect nbconvert/tests/test_nbconvertapp.py::TestNbConvertApp::test_webpdf_with_chromium" ``` At least with this syntax is very clear what tests are being deselected and how to add more tests later... I guess the optimal situation would be if upstream would mark these tests as requiring pyppeteer and skiping them if not available.