Skip to content

[stable34] test(ci): enforce PHPUnit time limits with sensible timeout values#60781

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/60739/stable34
Open

[stable34] test(ci): enforce PHPUnit time limits with sensible timeout values#60781
backportbot[bot] wants to merge 1 commit into
stable34from
backport/60739/stable34

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 27, 2026

Backport of PR #60739

@backportbot backportbot Bot requested a review from a team as a code owner May 27, 2026 16:25
@backportbot backportbot Bot requested review from Altahrim, ArtificialOwl, blizzz, icewind1991, kesselb, miaulalala and salmart-dev and removed request for a team May 27, 2026 16:25
@backportbot backportbot Bot added 3. to review Waiting for reviews tests Related to tests CI blocked labels May 27, 2026
@backportbot backportbot Bot added this to the Nextcloud 34 milestone May 27, 2026
PHPUnit's enforceTimeLimit was disabled, meaning the timeoutForSmallTests,
timeoutForMediumTests and timeoutForLargeTests config values had no effect.
Enable enforcement and set realistic limits: 60s/300s/600s for
small/medium/large, with a 300s default for unannotated tests.

Also clear disable_functions in the PHP development ini preset across all
PHPUnit workflows so pcntl_signal is available — without it the signal
handler that drives timeout enforcement cannot be registered.

Signed-off-by: Anna Larch <anna@nextcloud.com>
AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala force-pushed the backport/60739/stable34 branch from 778d3d9 to f793aa3 Compare May 28, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews blocked CI tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants