Modules syntax part 2#9
Open
wgrzeszczak wants to merge 107 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
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.
…ed scripts and package locks Delete 6 Playwright tests that depended on a /test-payment test app page which does not exist in this repo. The remaining two tests (invalid-transaction, missing-transaction-id) are the only ones that test the module's own pages. Also fixes seed script dependency ordering for user and chat modules, updates playwright.config.ts for user module, and adds missing package-lock.json files for data-export-api, oauth-facebook, oauth-google, openai, and payments modules.
Remove incorrect `await` on synchronous `expect()` calls in messaging.spec.ts.
Fix Chat constructor using bare `page`/`chatLocator` instead of `this.page`/`this.chatLocator`.
Add `waitFor({ state: 'visible' })` on message input in `isChatOpened` to prevent
race condition. Drop Firefox and WebKit projects from playwright.config.ts.
wgrzeszczak
force-pushed
the
modules-syntax-part-2
branch
from
4bbb106 to
6455536
Compare
wgrzeszczak
force-pushed
the
modules-syntax-part-2
branch
from
4be4cb1 to
8db404e
Compare
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