Modules syntax part 2#9
Open
wgrzeszczak wants to merge 103 commits into
Open
Conversation
Implement Playwright-based E2E tests (9 tests, all passing): - Checkout session creation and Stripe redirect validation - Payment page rendering and form submission - URL parameter preservation (success/cancel URLs) - Multiple payment attempts and transaction creation - API key verification and error handling Test application: - Simple payment form that creates transactions and redirects to Stripe checkout - Success/failure pages displaying transaction status - Debug page for troubleshooting Stripe API integration
- Remove all console.log statements from test specs - Remove emoji icons from debug-stripe.liquid page
Changes:
- Replace inline JSON with hash_assign pattern (9 files)
- Fix invalid filters: push→array_add, compact→array_compact (2 files)
- Fix single quotes in JSON strings (multiple files)
- Fix default: {} syntax (1 file)
Total: ~13 files modified in stripe_* commands and webhooks.
All 8 Playwright E2E tests in pos-module-payments-stripe have been marked as skipped because the required test application does not exist. The test app (tests/post_import/app/) with pages like /test-stripe-payment needs to be created before these tests can run.
Implement Playwright API tests for basic Stripe webhook processing and payment flows. Tests validate happy path scenarios for webhook handling, GraphQL operations, and successful payment completion.
Rename "E2E tests" workflow to "Tests" to better reflect that it runs both E2E tests (currently skipped) and API tests (active). Changes: - Rename workflow from "E2E tests" to "Tests" - Rename job from "test-e2e" to "run-tests" - Update all summary messages to use generic "Tests" terminology - Fix manual trigger handling: workflow_dispatch now always runs tests, bypassing duplicate action skip check
This reverts commit d6f7f19.
…dules into modules-syntax-part-2
Mirrors the oauth-github test structure: thin harness pages in
tests/post_import/ expose the module's Liquid helpers over HTTP.
Two tests:
- get_redirect_url builds a valid Google authorization URL with the
correct origin (accounts.google.com), path (/o/oauth2/v2/auth),
client_id, state, scope, response_type=code, and a redirect_uri
ending in /oauth/google/callback
- get_user_info returns { valid: false } when the OAuth callback
code is missing, without making an outbound Google API call
Add oauth-google to paths-filter, module-config, "all" dispatch array, and workflow_dispatch description.
Two smoke tests via thin harness pages in tests/post_import/: - get_redirect_url builds a valid Facebook authorization URL with the correct origin (www.facebook.com), path (/v22.0/dialog/oauth), client_id, state, scope=email,public_profile, and a redirect_uri ending in /oauth/facebook/callback - get_user_info returns { valid: false } when the OAuth callback code is missing, without making an outbound Facebook API call CI workflow updated: oauth-facebook added to paths-filter, module-config, "all" dispatch array, and description.
Four tests via harness pages in tests/post_import/: - transaction lifecycle: creates a transaction, verifies initial status is new, transitions to pending, verifies gateway_transaction_id is stored, and confirms a repeated pending update does not create a duplicate status record - status mapping: verifies expired, succeeded, and unknown payment statuses each map to the correct transaction status constant - validation: rejects transaction creation when payable_ids, amount_cents, and gateway are missing - gateway request receive: persists request metadata, payload, gateway_object_id, and stripe_account_name CI workflow updated: payments added to paths-filter, module-config, "all" dispatch array, and description.
Add missing dependency installs to seed.sh: common-styling and oauth_github must be installed before deploying the user module since it depends on both. Simplify the CI deploy-script for user: removing pos-modules files is no longer needed.
Add missing dependency installs to seed.sh: common-styling and user must be installed before deploying the chat module since it depends on both. Simplify the CI deploy-script for chat.
…dules into modules-syntax-part-2
Slashek
approved these changes
May 27, 2026
Replace the commented-out `pos-cli modules install payments` with `pos-cli modules install --dev`, which installs all dependencies declared in pos-module.json including payments and core.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Additional changes to syntax + tests