Fix failing GitHub CI pipeline #43
Closed
Conversation
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
The CI was failing because the bcmath PHP extension wasn't enabled in the GitHub Actions workflow files. Laravel Cashier requires the moneyphp/money package which depends on the bcmath extension. Changes: - Added bcmath extension to lint.yaml workflow - Added bcmath extension to test.yaml workflow
The CI tests were failing with "could not find driver" errors when trying to connect to SQLite databases. The pdo_sqlite PHP extension is required for Laravel's test database connection. Changes: - Added pdo_sqlite extension to lint.yaml workflow - Added pdo_sqlite extension to test.yaml workflow This fix addresses both the bcmath dependency and the SQLite PDO driver requirements for the CI pipeline.
Added missing steps to properly set up the SQLite database for testing: - Create database directory and file before running migrations - Run migrations before tests to ensure database schema exists This resolves the issue where tests were failing due to missing database tables.
The ignore pattern used lowercase 'components' but the actual directory is 'Components' (capital C). On case-sensitive filesystems (Linux/CI), this caused ESLint to parse the UI component files instead of ignoring them, resulting in parsing errors. Fixed by updating the ignore pattern to match the actual directory name: - resources/js/components/ui/**/* → resources/js/Components/ui/**/* This resolves the ~400 ESLint parsing errors in the CI pipeline.
Changed import.meta.glob to use eager loading ({ eager: true }), which
bundles all Vue pages at build time and includes them in the Vite manifest.
This allows tests to properly locate page components when rendering Inertia
responses.
Without eager loading, pages are loaded dynamically at runtime and aren't
included as separate entries in the manifest, causing "Unable to locate file
in Vite manifest" errors during test execution.
Also updated phpunit.xml to ensure proper test environment configuration.
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.
The CI was failing because the bcmath PHP extension wasn't enabled in the GitHub Actions workflow files. Laravel Cashier requires the moneyphp/money package which depends on the bcmath extension.
Changes: