test: Resolve testing comments (#1206)

Author: schloerke

.github/workflows/pytest.yaml

@@ -142,6 +142,13 @@ jobs:
         run: |
           make playwright-deploys SUB_FILE=". -vv"
 
+      - uses: actions/upload-artifact@v4
+        if: failure()
+        with:
+          name: "playright-examples-${{ matrix.os }}-${{ matrix.python-version }}-results"
+          path: test-results/
+          retention-days: 5
+
   playwright-deploys:
     needs: [playwright-deploys-precheck]
     if: github.event_name != 'release' && (github.event_name == 'push' || (github.event_name == 'pull_request' && startsWith(github.head_ref, 'deploy')))
@@ -171,7 +178,6 @@ jobs:
           make playwright-deploys SUB_FILE=". -vv"
 
       - name: Deploy apps and run tests (on `push` or on `pull_request` w/ `deploy**` branch)
-        if: ${{ github.event_name == 'push' || (github.event_name == 'pull_request' && startsWith(github.head_ref, 'deploy')) }}
         env:
           DEPLOY_APPS: "true"
           DEPLOY_CONNECT_SERVER_URL: "https://rsc.radixu.com/"

tests/playwright/conftest.py

@@ -55,6 +55,8 @@ def session_page(browser: BrowserContext) -> Page:
 # This must be done before each test
 def page(session_page: Page) -> Page:
     session_page.goto("about:blank")
+    # Reset screen size to 1080p
+    session_page.set_viewport_size({"width": 1920, "height": 1080})
     return session_page