Open
Conversation
- Add basic Docker integration tests (run-integration-tests.sh) - Validates Docker stack startup - Checks login page accessibility and HTTP status - Verifies login form presence - Add Playwright E2E test suite - Login tests: valid/invalid credentials, protected pages - Items tests: create, update, verify in inventory table - Customers tests: create with details, search, table verification - Sales tests: full sale flow with items, customers, payment, and receipt validation - Configure Playwright with multi-browser support (Chrome, Firefox) - Add GitHub Actions workflow for CI/CD - Runs on push/PR to master - Includes both basic and Playwright tests - Uploads screenshots, traces, and logs on failure - Organize tests in integration-tests/ directory - Update package.json with test scripts - Include comprehensive documentation This provides automated testing for core POS workflows including item management, customer management, and complete sales transactions with receipt generation verification.
objecttothis
previously approved these changes
Mar 6, 2026
The --wait flag causes CI to fail when init containers (like sqlscript) exit successfully with code 0. The workflow already has a dedicated 'Wait for Application' step that polls the application endpoint. Also remove redundant webServer config from playwright.config.ts since the workflow handles container startup separately.
Member
Author
CI Fix AppliedThe test failures were caused by Root Cause:
Fix:
The tests should now run properly once the CI re-runs. |
…yntax error - Add 'needs: integration' and 'if: always()' so playwright job runs even when integration job fails, ensuring screenshots get uploaded - Fix npm run test command (remove incorrect cd into integration-tests) - Fix JavaScript error: navigationOption used before declaration - Add explicit outputDir and outputFolder in playwright config
objecttothis
approved these changes
Mar 8, 2026
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
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.
Add basic Docker integration tests (run-integration-tests.sh)
Add Playwright E2E test suite
Configure Playwright with multi-browser support (Chrome, Firefox)
Add GitHub Actions workflow for CI/CD
Organize tests in integration-tests/ directory
Update package.json with test scripts
Include comprehensive documentation
This provides automated testing for core POS workflows including item management, customer management, and complete sales transactions with receipt generation verification.