Add e2e tests to pos-module-payments-example-gateway

.github/workflows/lint.yml

@@ -1,14 +1,16 @@
 name: Lint modules
 
 on:
-  pull_request:
-    branches: [ master ]
-    paths-ignore:
-      - "**/README.md"
   push:
     paths-ignore:
       - "**/README.md"
   workflow_dispatch:
+    inputs:
+      modules:
+        description: 'Modules to lint (comma-separated or "all" for all modules)'
+        required: false
+        default: 'all'
+        type: string
 
 jobs:
   pre_job:
@@ -21,7 +23,7 @@ jobs:
         with:
           github_token: ${{ github.token }}
           paths_ignore: '["**/README.md"]'
-          do_not_skip: '["push"]'
+          do_not_skip: '["push", "workflow_dispatch"]'
 
   detect-changes:
     needs: pre_job
@@ -34,7 +36,10 @@ jobs:
 
       - uses: dorny/paths-filter@v3
         id: filter
+        if: github.event_name != 'workflow_dispatch'
         with:
+          base: ${{ github.event.before }}
+          ref: ${{ github.event.after }}
           filters: |
             payments:
               - 'pos-module-payments/**'
@@ -68,11 +73,29 @@ jobs:
       - name: Set matrix for changed modules
         id: set-matrix
         run: |
-          # Extract module names where filter output is "true"
-          modules=$(echo '${{ toJSON(steps.filter.outputs) }}' | jq -c '[to_entries[] | select(.value == "true") | .key]')
+          # Check if this is a manual trigger
+          if [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
+            echo "Manual trigger detected"
+            manual_input="${{ github.event.inputs.modules }}"
+            echo "Manual input: $manual_input"
+
+            if [ "$manual_input" = "all" ] || [ -z "$manual_input" ]; then
+              # Lint all modules
+              modules='["payments","user","chat","common-styling","tests","core","oauth-facebook","oauth-github","oauth-google","openai","reports","data-export-api","payments-stripe","payments-example-gateway"]'
+            else
+              # Parse comma-separated list, strip pos-module- prefix, and create array
+              modules=$(echo "$manual_input" | jq -R 'split(",") | map(gsub("^\\s+|\\s+$"; "") | gsub("^pos-module-"; ""))')
+            fi
+            echo "Manual trigger modules: $modules"
+          else
+            # Automatic detection from git changes
+            changes='${{ steps.filter.outputs.changes }}'
+            echo "Detected changes: $changes"
+            modules="$changes"
+            echo "Changed modules for linting: $modules"
+          fi
 
           echo "matrix=$modules" >> $GITHUB_OUTPUT
-          echo "Changed modules for linting: $modules"
 
   lint-platformos-check:
     needs: detect-changes
@@ -168,3 +191,41 @@ jobs:
         run: |
           echo "platformos-check failed — marking job as failed"
           exit 1
+
+  conclusion:
+    needs: [detect-changes, lint-platformos-check]
+    if: always()
+    runs-on: ubuntu-latest
+    steps:
+      - name: Generate workflow summary
+        run: |
+          if [ "${{ needs.detect-changes.outputs.changed-modules }}" = "[]" ] || [ "${{ needs.detect-changes.result }}" = "skipped" ]; then
+            echo "## Linting - Skipped" >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+            if [ "${{ needs.detect-changes.result }}" = "skipped" ]; then
+              echo "Workflow was skipped by duplicate action check." >> $GITHUB_STEP_SUMMARY
+            elif [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
+              echo "Manual trigger with no matching modules selected." >> $GITHUB_STEP_SUMMARY
+            else
+              echo "No modules were changed in this push." >> $GITHUB_STEP_SUMMARY
+            fi
+          else
+            if [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
+              echo "## Linting - Completed (Manual Trigger)" >> $GITHUB_STEP_SUMMARY
+            else
+              echo "## Linting - Completed" >> $GITHUB_STEP_SUMMARY
+            fi
+            echo "" >> $GITHUB_STEP_SUMMARY
+            echo "Linting ran for the following modules:" >> $GITHUB_STEP_SUMMARY
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            echo '${{ needs.detect-changes.outputs.changed-modules }}' | jq -r '.[] | "- " + .' >> $GITHUB_STEP_SUMMARY
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+            if [ "${{ needs.lint-platformos-check.result }}" = "success" ]; then
+              echo "Result: All checks passed" >> $GITHUB_STEP_SUMMARY
+            elif [ "${{ needs.lint-platformos-check.result }}" = "skipped" ]; then
+              echo "Result: Linting was skipped" >> $GITHUB_STEP_SUMMARY
+            else
+              echo "Result: Some checks failed - see job output for details" >> $GITHUB_STEP_SUMMARY
+            fi
+          fi

.github/workflows/test-e2e.yml

@@ -1,13 +1,15 @@
 name: E2E tests
 on:
-  pull_request:
-    branches: [ master ]
-    paths-ignore:
-      - "**/README.md"
   push:
     paths-ignore:
       - "**/README.md"
   workflow_dispatch:
+    inputs:
+      modules:
+        description: 'Modules to test (comma-separated: user, chat, common-styling, payments-example-gateway, or "all")'
+        required: false
+        default: 'all'
+        type: string
 
 jobs:
   pre_job:
@@ -20,7 +22,7 @@ jobs:
         with:
           github_token: ${{ github.token }}
           paths_ignore: '["**/README.md"]'
-          do_not_skip: '["push"]'
+          do_not_skip: '["push", "workflow_dispatch"]'
 
   detect-changes:
     needs: pre_job
@@ -33,14 +35,19 @@ jobs:
 
       - uses: dorny/paths-filter@v3
         id: filter
+        if: github.event_name != 'workflow_dispatch'
         with:
+          base: ${{ github.event.before }}
+          ref: ${{ github.event.after }}
           filters: |
             user:
               - 'pos-module-user/**'
             chat:
               - 'pos-module-chat/**'
             common-styling:
               - 'pos-module-common-styling/**'
+            payments-example-gateway:
+              - 'pos-module-payments-example-gateway/**'
 
       - name: Set matrix for changed modules
         id: set-matrix
@@ -65,17 +72,45 @@ jobs:
               "path": "pos-module-common-styling",
               "deploy-script": "pos-cli data clean --include-schema --auto-confirm\npos-cli deploy",
               "test-commands": "npm run pw-tests"
+            },
+            "payments-example-gateway": {
+              "module": "payments-example-gateway",
+              "path": "pos-module-payments-example-gateway",
+              "deploy-script": "npm run test:setup:local\n./tests/data/seed/seed.sh",
+              "test-commands": "npm run pw-tests"
             }
           }
           EOF
 
-          # Extract changed modules and map to their configurations
-          modules=$(echo '${{ toJSON(steps.filter.outputs) }}' | \
-            jq -c --slurpfile config /tmp/module-config.json \
-            '[to_entries[] | select(.value == "true") | .key as $m | $config[0][$m] | select(. != null)]')
+          # Check if this is a manual trigger
+          if [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
+            echo "Manual trigger detected"
+            manual_input="${{ github.event.inputs.modules }}"
+            echo "Manual input: $manual_input"
+
+            if [ "$manual_input" = "all" ] || [ -z "$manual_input" ]; then
+              # Test all modules
+              modules=$(jq -c '[.user, .chat, ."common-styling", ."payments-example-gateway"]' /tmp/module-config.json)
+            else
+              # Parse comma-separated list, strip pos-module- prefix, and map to configurations
+              modules=$(echo "$manual_input" | \
+                jq -R 'split(",") | map(gsub("^\\s+|\\s+$"; "") | gsub("^pos-module-"; ""))' | \
+                jq -c --slurpfile config /tmp/module-config.json \
+                'map(. as $m | $config[0][$m] | select(. != null))')
+            fi
+            echo "Manual trigger modules: $modules"
+          else
+            # Automatic detection from git changes
+            changes='${{ steps.filter.outputs.changes }}'
+            echo "Detected changes: $changes"
+
+            # Extract changed modules and map to their configurations
+            modules=$(jq -nc --argjson changes "$changes" --slurpfile config /tmp/module-config.json \
+              '$changes | map(. as $m | $config[0][$m] | select(. != null))')
+            echo "Changed modules matrix: $modules"
+          fi
 
           echo "matrix=$modules" >> $GITHUB_OUTPUT
-          echo "Changed modules matrix: $modules"
 
   test-e2e:
     needs: detect-changes
@@ -148,3 +183,41 @@ jobs:
           method: release
           repository-url: ${{ vars.CI_PS_REPOSITORY_URL }}
           pos-ci-repo-token: ${{ secrets.POS_CI_PS_REPO_ACCESS_TOKEN }}
+
+  conclusion:
+    needs: [detect-changes, test-e2e]
+    if: always()
+    runs-on: ubuntu-latest
+    steps:
+      - name: Generate workflow summary
+        run: |
+          if [ "${{ needs.detect-changes.outputs.changed-modules }}" = "[]" ] || [ "${{ needs.detect-changes.result }}" = "skipped" ]; then
+            echo "## E2E Tests - Skipped" >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+            if [ "${{ needs.detect-changes.result }}" = "skipped" ]; then
+              echo "Workflow was skipped by duplicate action check." >> $GITHUB_STEP_SUMMARY
+            elif [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
+              echo "Manual trigger with no matching modules selected." >> $GITHUB_STEP_SUMMARY
+            else
+              echo "No modules with E2E tests were changed in this push." >> $GITHUB_STEP_SUMMARY
+            fi
+          else
+            if [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
+              echo "## E2E Tests - Completed (Manual Trigger)" >> $GITHUB_STEP_SUMMARY
+            else
+              echo "## E2E Tests - Completed" >> $GITHUB_STEP_SUMMARY
+            fi
+            echo "" >> $GITHUB_STEP_SUMMARY
+            echo "Tests ran for the following modules:" >> $GITHUB_STEP_SUMMARY
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            echo '${{ needs.detect-changes.outputs.changed-modules }}' | jq -r '.[] | "- " + .module' >> $GITHUB_STEP_SUMMARY
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+            if [ "${{ needs.test-e2e.result }}" = "success" ]; then
+              echo "Result: All tests passed" >> $GITHUB_STEP_SUMMARY
+            elif [ "${{ needs.test-e2e.result }}" = "skipped" ]; then
+              echo "Result: Tests were skipped" >> $GITHUB_STEP_SUMMARY
+            else
+              echo "Result: Some tests failed - check job output for details" >> $GITHUB_STEP_SUMMARY
+            fi
+          fi

pos-module-chat/package.json

@@ -5,7 +5,7 @@
     "install: playwright": "npx playwright install"
   },
   "devDependencies": {
-    "@playwright/test": "^1.49.1",
+    "@playwright/test": "^1.58.2",
     "@types/node": "^22.10.6"
   }
 }