Remove pixel-perfect screenshot tests

Per feedback from other maintainers, these tests will just be a waste of time as we make numerous necessary design fixes and improvements.

Author: theodorejb

.github/workflows/preview-tests.yml

@@ -4,8 +4,8 @@ on:
     types: [ labeled ]
 
 jobs:
-  tests_visual:
-    name: "Visual Tests"
+  tests_e2e:
+    name: "E2E Tests"
     runs-on: "ubuntu-latest"
     if: "github.repository_owner == 'php' && contains(github.event.pull_request.labels.*.name, 'Status: Preview Allowed')"
 
@@ -77,8 +77,8 @@ jobs:
       - name: "Install Playwright"
         run: "npx playwright install"
 
-      - name: "Run visual tests"
-        run: "make tests_visual"
+      - name: "Run E2E tests"
+        run: "make tests_e2e"
 
       - name: Upload playwright report
         uses: actions/upload-artifact@v7

.github/workflows/update-screenshots.yaml

@@ -26,14 +26,9 @@ tests: vendor ## Runs unit and end-to-end tests with phpunit/phpunit
 	vendor/bin/phpunit --configuration=tests/phpunit.xml --testsuite=end-to-end;
 	tests/server stop
 
-tests_visual:
+tests_e2e:
 	tests/server start;
-	npx playwright test --workers=$(CORES)
-	tests/server stop
-
-tests_update_snapshots:
-	tests/server start;
-	npx playwright test --update-snapshots
+	npx playwright test
 	tests/server stop
 
 vendor: composer.json composer.lock

Makefile

@@ -30,11 +30,6 @@ export default defineConfig({
     timeout: 0,
 
     projects: [
-        {
-            name: 'chromium',
-            use: {...devices['Desktop Chrome']},
-            testDir: './tests/Visual',
-        },
         {
             name: 'End-to-End Chromium',
             use: {...devices['Desktop Chrome']},