Re: [PR REVIEW] Jupyter notebook updates

[tornaria <tornaria@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 1252 bytes --]

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.