[PR #3751] [MERGED] Parallel Testing Support #6256

New Issue

OVERLORD · 2026-02-05T10:27:49+03:00

OVERLORD commented

2026-02-05 10:27:49 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/3751
Author: @ssddanbrown
Created: 9/26/2022
Status: ✅ Merged
Merged: 9/27/2022
Merged by: @ssddanbrown

Base: development ← Head: parallel_testing

📝 Commits (2)

e18033e Added initial support for parallel testing
f21669c Cleaned testing service provider usage

📊 Changes

5 files changed (+61 additions, -13 deletions)

View changed files

📝 composer.json (+3 -0)
📝 readme.md (+2 -7)
📝 tests/TestCase.php (+24 -0)
➕ tests/TestServiceProvider.php (+26 -0)
📝 tests/ThemeTest.php (+6 -6)

📄 Description

Adds support for Laravel's built-in parallel testing support.

Can be ran via php artisan test -p. Will create a test database for each thread.

Todo

Update seeding technique to only seed if needed, not every run.
Move AppServiceProvider to a test-only place if possible? Not a fan of test-specific code living there.
Document in readme?
Revert change to resources/js/wysiwyg/config.js

_{🔄 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/3751 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 9/26/2022 **Status:** ✅ Merged **Merged:** 9/27/2022 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `parallel_testing` --- ### 📝 Commits (2) - [`e18033e`](https://github.com/BookStackApp/BookStack/commit/e18033ec1ae181a8977d23d14090d0706f3cc05b) Added initial support for parallel testing - [`f21669c`](https://github.com/BookStackApp/BookStack/commit/f21669c0c966f3dadeac2024a382b8a7cd831a8a) Cleaned testing service provider usage ### 📊 Changes **5 files changed** (+61 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `composer.json` (+3 -0) 📝 `readme.md` (+2 -7) 📝 `tests/TestCase.php` (+24 -0) ➕ `tests/TestServiceProvider.php` (+26 -0) 📝 `tests/ThemeTest.php` (+6 -6) </details> ### 📄 Description Adds support for Laravel's built-in parallel testing support. Can be ran via `php artisan test -p`. Will create a test database for each thread. ### Todo - [x] Update seeding technique to only seed if needed, not every run. - [x] Move `AppServiceProvider` to a test-only place if possible? Not a fan of test-specific code living there. - [x] Document in readme? - [x] Revert change to `resources/js/wysiwyg/config.js` --- <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:27:49 +03:00

OVERLORD closed this issue

2026-02-05 10:27:51 +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#6256