test(build): fix the installation tests of the sdist

- The wheel jobs run test_install only; the sdist job runs test_install_sdist.
  Each job builds the one artifact its test installs.
- The sdist job builds the wheel first, so that the checkout has a driver
  with Node.js for install-deps.
- The client subprocesses of the tests run from tmp_path, so that they import
  the installed package and not the checkout.
- The sdist test installs with --no-binary :all:, the pip form of poetry's
  installer.no-binary policy of the report.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Author: dgozman

.github/workflows/ci.yml

@@ -131,7 +131,7 @@ jobs:
     - name: Test Reference count
       run: pytest tests/test_reference_count_async.py --browser=${{ matrix.browser }}
     - name: Test Wheel Installation
-      run: pytest tests/test_installation.py --browser=${{ matrix.browser }}
+      run: pytest tests/test_installation.py::test_install --browser=${{ matrix.browser }}
     - name: Test Sync API
       if: runner.os != 'Linux'
       run: pytest tests/sync --browser=${{ matrix.browser }} -n ${{ env.WORKERS }} -p pytest_dot_reporter -rfE --color=yes
@@ -227,12 +227,16 @@ jobs:
         pip install -e .
     - name: Get Playwright
       run: python scripts/upstream.py checkout
-    - name: Build the driver and the sdist
+    # The wheel build puts the driver with Node.js into the checkout, for the
+    # install-deps step below. The test installs the sdist in its own venv.
+    - name: Build the driver and the wheel
+      run: python -m build --wheel
+    - name: Build the sdist
       run: python -m build --sdist
     - name: Install browser dependencies
-      run: PLAYWRIGHT_NODEJS_PATH=$(which node) python -m playwright install-deps chromium
+      run: python -m playwright install-deps chromium
     - name: Test the sdist installation
-      run: pytest tests/test_installation.py -k test_install_sdist --browser=chromium
+      run: pytest tests/test_installation.py::test_install_sdist --browser=chromium
 
   test_examples:
     name: Examples

tests/test_installation.py

@@ -31,12 +31,16 @@ def _create_venv(env_dir: Path) -> str:
 
 
 def _run_client(python: str, tmp_path: Path, browser_name: str, environ: dict) -> None:
+    # cwd=tmp_path: from the repository root, "-m playwright" imports the checkout
+    # instead of the installed package.
     subprocess.check_output(
-        [python, "-m", "playwright", "install", browser_name], env=environ
+        [python, "-m", "playwright", "install", browser_name],
+        cwd=tmp_path,
+        env=environ,
     )
     shutil.copyfile(ROOT / "tests" / "assets" / "client.py", tmp_path / "main.py")
     subprocess.check_output(
-        [python, str(tmp_path / "main.py"), browser_name], env=environ
+        [python, str(tmp_path / "main.py"), browser_name], cwd=tmp_path, env=environ
     )
     assert (tmp_path / f"{browser_name}.png").exists()
 
@@ -66,7 +70,12 @@ def test_install_sdist(tmp_path: Path, browser_name: str) -> None:
     assert node, "node must be in PATH"
     python = _create_venv(tmp_path / "env")
     sdist = list((ROOT / "dist").glob("playwright-*.tar.gz"))[0]
-    subprocess.check_output([python, "-m", "pip", "install", str(sdist)])
+    # No wheels at all, as poetry's installer.no-binary=:all: in the report: the
+    # dependencies come from source too. The isolated build environment still
+    # takes wheels: pip does not pass the policy on to it, and neither does poetry.
+    subprocess.check_output(
+        [python, "-m", "pip", "install", "--no-binary", ":all:", str(sdist)]
+    )
     driver_dir = Path(
         subprocess.check_output(
             [