[PR #2522] [MERGED] Docker: Fix PHP tests #6012

New Issue

OVERLORD · 2026-02-05T10:22:34+03:00

OVERLORD commented

2026-02-05 10:22:34 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/2522
Author: @Abijeet
Created: 1/31/2021
Status: ✅ Merged
Merged: 3/21/2021
Merged by: @ssddanbrown

Base: master ← Head: docker-tests

📝 Commits (2)

ba2033a Docker: Fix permission with node service by adding user key
a0c605f Docker: Fix PHP tests

📊 Changes

6 files changed (+29 additions, -8 deletions)

View changed files

📝 app/Config/database.php (+2 -1)
📝 dev/docker/entrypoint.app.sh (+1 -0)
➕ dev/docker/init.db/01.sql (+5 -0)
📝 docker-compose.yml (+3 -2)
📝 phpunit.xml (+3 -2)
📝 readme.md (+15 -3)

📄 Description

This creates another mysql_testing database during db service setup

Replace server with env tags in phpunit.xml in order to force
override certain parameters when tests are run. See:
sebastianbergmann/phpunit#2353 for more
information.

Rename primary developer Docker database from bookstack-test to
bookstack-dev. bookstack-test is used as the mysql_testing database

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/2522 **Author:** [@Abijeet](https://github.com/Abijeet) **Created:** 1/31/2021 **Status:** ✅ Merged **Merged:** 3/21/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `docker-tests` --- ### 📝 Commits (2) - [`ba2033a`](https://github.com/BookStackApp/BookStack/commit/ba2033a8fbb1ab3607599d4bc82456a4aa6bfe95) Docker: Fix permission with node service by adding user key - [`a0c605f`](https://github.com/BookStackApp/BookStack/commit/a0c605faaeb06cdd1426d48bfbe643954b4be582) Docker: Fix PHP tests ### 📊 Changes **6 files changed** (+29 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `app/Config/database.php` (+2 -1) 📝 `dev/docker/entrypoint.app.sh` (+1 -0) ➕ `dev/docker/init.db/01.sql` (+5 -0) 📝 `docker-compose.yml` (+3 -2) 📝 `phpunit.xml` (+3 -2) 📝 `readme.md` (+15 -3) </details> ### 📄 Description This creates another mysql_testing database during db service setup Replace server with env tags in phpunit.xml in order to force override certain parameters when tests are run. See: sebastianbergmann/phpunit#2353 for more information. Rename primary developer Docker database from bookstack-test to bookstack-dev. bookstack-test is used as the mysql_testing database --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-05 10:22:34 +03:00

OVERLORD closed this issue

2026-02-05 10:22:36 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#6012