Contribution Date
Contribution Project
Contribution Details
Refactor: consolidate workflow, fix SQLite permissions and debug output
This commit consolidates the Docker build and test workflow into a single file (`docker-buildx.yml`), removing the redundant `test-drupal.yml`. It ensures the build is tested before pushing.
Additionally, it fixes several issues with the Drupal installation and testing:
- **SQLite Installation:**
- `tests/install-drupal.sh`: Corrected the Drush SQLite connection string to remove `localhost` (`sqlite://sites/default/files/.ht.sqlite`).
- Added logic to parse `drush status` JSON output to find the exact database path and `chmod 777` its directory, resolving write permission errors.
- **Workflow Logic:**
- `docker-buildx.yml`: Passes explicit version constraints (`^10`, `^11`) to the install script.
- Cleans `drupal-root` before starting containers.
- **Debugging:**
- `tests/verify-drupal.sh`: Updated to log the permissions of the SQLite database file and its directory specifically, rather than a generic `ls -la`.
- **Image Compatibility:**
- `php8/apache-bookworm/Dockerfile`: Added `SQLITE_ENABLE_COLUMN_METADATA=1` to fix Drupal compatibility.
Co-authored-by: hussainweb <1040271+hussainweb@users.noreply.github.com>
Contribution Issue Link
Files count
0
Patches count
1