Add Settings API and more #331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open

superdav42 wants to merge 32 commits into main from tweask-again

.github/workflows/e2e.yml

-Original file line number
+Diff line change
@@ Expand Up / @@ -120,66 +120,24 @@ jobs: @@
               echo "Base URL: http://localhost:8889"
               cat cypress.env.json 2>/dev/null || echo "❌ cypress.env.json not found"
-          - name: Run Setup Wizard Test (Must Run First)
+          - name: Run Setup Test (Must Run First)
             id: setup-test
-            continue-on-error: true
             run: |
-              echo "=== Starting Setup Wizard Test ==="
-              # Don't exit on errors - we want to capture the result
+              echo "=== Starting Setup Test ==="
               npx cypress run \
                 --config-file cypress.config.test.js \
-                --spec "tests/e2e/cypress/integration/setup-wizard-complete.spec.js" \
-                --browser ${{ matrix.browser }} \
-                --reporter spec \
-                --reporter-options "verbose=true" \
-                --env "CYPRESS_CRASH_REPORTS=0" \
-                --record false \
-                --video true \
-                --screenshot-on-failure true
-              CYPRESS_EXIT_CODE=$?
-              if [ $CYPRESS_EXIT_CODE -eq 0 ]; then
-                echo "✅ Setup wizard test passed successfully"
-                echo "setup_success=true" >> $GITHUB_OUTPUT
-              else
-                echo "❌ Setup wizard test failed with exit code $CYPRESS_EXIT_CODE"
-                echo "Capturing detailed failure information..."
-                echo "=== WordPress State ==="
-                curl -I http://localhost:8889 || echo "WordPress not responding"
-                echo "=== Test Files ==="
-                ls -la tests/e2e/cypress/screenshots/ 2>/dev/null || echo "No screenshots directory"
-                ls -la tests/e2e/cypress/videos/ 2>/dev/null || echo "No videos directory"
-                echo "=== Docker Logs ==="
-                docker logs $(docker ps -q --filter "name=tests-wordpress") 2>&1 | tail -10 || echo "No Docker logs"
-                echo "=== File System ==="
-                ls -la tests/e2e/cypress/integration/setup-wizard-complete.spec.js || echo "Setup test file missing"
-                echo "setup_success=false" >> $GITHUB_OUTPUT
-              fi
-          - name: Verify Setup Completed Successfully
-            if: success()
-            run: |
-              echo "=== Verifying Setup Wizard Completion ==="
-              # Test for setup completion indicators
-              curl -s "http://localhost:8889/wp-admin/network/" | grep -i "ultimo\|dashboard" && echo "✅ Setup appears successful" || echo "⚠️  Setup verification unclear"
+                --spec "tests/e2e/cypress/integration/000-setup.spec.js" \
+                --browser ${{ matrix.browser }}
           - name: Run Checkout Tests (After Setup)
-            if: always()  # Run checkout tests regardless of setup wizard result for debugging
-            continue-on-error: true
+            id: checkout-tests
             run: |
+              set +e
               echo "=== Starting Checkout Tests ==="
-              echo "Setup wizard result: ${GITHUB_OUTPUT}"
-              echo "Setup success flag: ${{ steps.setup-test.outputs.setup_success }}"
-              echo "Running checkout test suite (setup wizard success not required for debugging)..."
-              # Run all checkout tests in sequence
               CHECKOUT_TESTS=(
-                "tests/e2e/cypress/integration/checkout-registration.spec.js"
-                "tests/e2e/cypress/integration/checkout-validation.spec.js"
-                "tests/e2e/cypress/integration/checkout-scenarios.spec.js"
-                "tests/e2e/cypress/integration/checkout-confirmation.spec.js"
+                "tests/e2e/cypress/integration/010-manual-checkout-flow.spec.js"
+                "tests/e2e/cypress/integration/020-free-trial-flow.spec.js"
               )
               TOTAL_FAILURES=0
@@ Expand All / @@ -190,13 +148,7 @@ jobs: @@
                 npx cypress run \
                   --config-file cypress.config.test.js \
                   --spec "$TEST_SPEC" \
-                  --browser ${{ matrix.browser }} \
-                  --reporter spec \
-                  --reporter-options "verbose=true" \
-                  --env "CYPRESS_CRASH_REPORTS=0" \
-                  --record false \
-                  --video true \
-                  --screenshot-on-failure true
+                  --browser ${{ matrix.browser }}
                 CYPRESS_EXIT_CODE=$?
@@ Expand All / @@ -209,14 +161,12 @@ jobs: @@
               done
               if [ $TOTAL_FAILURES -gt 0 ]; then
-                echo "❌ $TOTAL_FAILURES checkout tests failed"
-                echo "Capturing failure information..."
-                ls -la tests/e2e/cypress/screenshots/ 2>/dev/null || echo "No screenshots"
-                ls -la tests/e2e/cypress/videos/ 2>/dev/null || echo "No videos"
-              else
-                echo "✅ All checkout tests passed successfully!"
+                echo "❌ $TOTAL_FAILURES checkout test(s) failed"
+                exit 1
               fi
+              echo "✅ All checkout tests passed!"
           - name: Fix permissions for Cypress output
             if: always()
             run: sudo chown -R $USER:$USER tests/e2e/cypress
@@ Expand All / @@ -237,12 +187,6 @@ jobs: @@
               name: cypress-videos-${{ matrix.php }}-${{ matrix.browser }}
               path: tests/e2e/cypress/videos
-          - name: Fail job if tests failed
-            if: failure()
-            run: |
-              echo "❌ One or more e2e tests failed."
-              exit 1
           - name: Stop WordPress Environment
             if: always()
             run: npm run env:stop

.wp-env.json

-Original file line number
+Diff line change
@@ Expand Up / @@ -7,15 +7,17 @@ @@
             "."
           ],
           "mappings": {
-            "wp-content/mu-plugins/email-smtp-dev.php": "./mu-plugins/email-smtp-dev/email-smtp-dev.php"
+            "wp-content/mu-plugins/email-smtp-dev.php": "./mu-plugins/email-smtp-dev/email-smtp-dev.php",
+            "wp-content/sunrise.php": "./sunrise.php"
           },
           "config": {
             "WP_DEBUG": true,
             "WP_DEBUG_DISPLAY": true,
             "WP_DEBUG_LOG": true,
             "SCRIPT_DEBUG": true,
             "WP_MEMORY_LIMIT": "256M",
-            "WP_MAX_MEMORY_LIMIT": "512M"
+            "WP_MAX_MEMORY_LIMIT": "512M",
+            "SUNRISE": true
           }
         },
         "tests": {
@@ Expand All / @@ -24,15 +26,17 @@ @@
             "."
           ],
           "mappings": {
-            "wp-content/mu-plugins/email-smtp-test.php": "./mu-plugins/email-smtp-test/email-smtp-test.php"
+            "wp-content/mu-plugins/email-smtp-test.php": "./mu-plugins/email-smtp-test/email-smtp-test.php",
+            "wp-content/sunrise.php": "./sunrise.php"
           },
           "config": {
             "WP_DEBUG": true,
             "WP_DEBUG_DISPLAY": false,
             "WP_DEBUG_LOG": true,
             "SCRIPT_DEBUG": true,
             "WP_MEMORY_LIMIT": "256M",
-            "WP_MAX_MEMORY_LIMIT": "512M"
+            "WP_MAX_MEMORY_LIMIT": "512M",
+            "SUNRISE": true
           }
         }
       }
@@ Expand Down @@

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add Settings API and more #331

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Uh oh!

Uh oh!

Add Settings API and more #331

Are you sure you want to change the base?

Uh oh!

Add Settings API and more #331

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Uh oh!