Merge pull request #2652 from asfadmin/lily/visualregressiontesting

Only update snapshots on successful tests in update snapshots action

Author: lllarsen

.github/workflows/update-snapshots.yaml

@@ -46,7 +46,7 @@ jobs:
       - name: Install angular cli
         run: npm install -g @angular/cli
       - name: Start server and record snapshots
-        run: ng serve --host=127.0.0.1 & npx wait-on http://127.0.0.1:4200 && npx playwright test --update-snapshots --reporter=list --grep '@visual' || true
+        run: ng serve --host=127.0.0.1 & npx wait-on http://127.0.0.1:4200 && npx playwright test --update-snapshots --reporter=list --grep '@visual'
       - uses: stefanzweifel/git-auto-commit-action@v7
         with:
           commit_message: "[CI] Update Snapshots"