fix(workflow): update Playwright test environment variables and artifact upload step

Author: pauldotyu

.github/workflows/test-e2e.yaml

@@ -80,14 +80,16 @@ jobs:
         run: npx playwright test
         working-directory: tests
         env:
+          PLAYWRIGHT_SERVICE_URL: ${{ secrets.PLAYWRIGHT_SERVICE_URL }}
           STORE_ADMIN_URL: ${{ steps.kubectl_get_service.outputs.STORE_ADMIN_IP }}
           STORE_FRONT_URL: ${{ steps.kubectl_get_service.outputs.STORE_FRONT_IP }}
+          CI: true
 
-      - uses: actions/upload-artifact@v3
-        if: always()
+      - uses: actions/upload-artifact@v4
+        if: ${{ !cancelled() }}
         with:
-          name: playwright-report-store-admin
-          path: tests/playwright-report
+          name: playwright-report
+          path: tests/playwright-report/
           retention-days: 30
 
       - name: Destroy environment