Adopt playwright

[brichet]

Since pyppeteer is now unmaintained we should adopt playwright instead.

[brichet]

The workaround commit is to fix the tests when the chromium browser is not available.

This test worked only if pytest do not capture the output (pytest -s).
I don't understand why, but os.path.isfile(chromium.executable_path) was True when capturing the output but False otherwise...

[brichet]

I have no clue on this test failure too (occurs locally also):
https://github.com/jupyter/nbconvert/actions/runs/5402860958/jobs/9814806087?pr=2013#step:5:379

Don't know why it used to pass as exporter.from_file(f, resources={}) is supposed to return a tuple

nbconvert/nbconvert/exporters/exporter.py

Lines 203 to 205 in 50b86f8

	def from_file(
	self, file_stream: t.Any, resources: t.Optional[dict] = None, **kw: t.Any
	) -> t.Tuple[NotebookNode, dict]:

but exporter.from_notebook_node(nb) expects a NotebookNode (it lead to exporter.html.from_notebook_node(nb)):

nbconvert/nbconvert/exporters/html.py

Lines 227 to 229 in 50b86f8

	def from_notebook_node( # type:ignore
	self, nb: NotebookNode, resources: Optional[Dict] = None, **kw: Any
	) -> Tuple[str, Dict]:

[brichet]

Since it fails with python<3.8, I wonder if we should drop support to Python 3.7 (easy fix).
Playwright will drop support to it in the next release: microsoft/playwright-python#1980

[blink1073]

I'm removing Python 3.7 support in #2008. I'm updating this branch to pick up those changes.

[blink1073]

Thanks again!

[brichet]

Thanks @blink1073 for fixing the tests

[fcollonval]

It seems that that option was lost in the upgrade process. As the option is available in playwright, I guess it will be better to restore it to increase backward compatibility.

cc @brichet

[brichet]

Thanks @fcollonval. I opened #2022