Skip to content

Test build - do not merge - #9119

Open
nolanpro wants to merge 1 commit into
developfrom
test-build
Open

nolanpro wants to merge 1 commit into
developfrom
test-build

Conversation

@nolanpro

@nolanpro nolanpro commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

ci:skip-build
ci:phpunit-coverage
.

@cursor

cursor Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Changes are limited to test infrastructure and a README typo, with no production runtime behavior changes.

Overview
Test helper change: runScheduledTasks() in ProcessTestingTrait no longer drives Laravel’s Schedule to execute due bpmn:timer events. It now calls TaskSchedulerManager::scheduleTasks() and evaluateConditionals() directly—the same steps as the bpmn:timer artisan command—so feature tests stay in-process.

This avoids redundant timer work and documents/fixes long hangs under RefreshDatabase, where re-running the scheduled command while the test transaction is open could hit InnoDB lock wait timeouts (~50s).

README: A stray . was added on the first line (looks incidental given the PR title).

Reviewed by Cursor Bugbot for commit 89bdc8d. Bugbot is set up for automated code reviews on this repo. Configure here.

@decisions-sonarqube

Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
53 New issues
E Security Rating on New Code (required ≥ A)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant