Skip to content

Test events while process isolation#6291

Closed
staabm wants to merge 2 commits intosebastianbergmann:mainfrom
staabm:test-events
Closed

Test events while process isolation#6291
staabm wants to merge 2 commits intosebastianbergmann:mainfrom
staabm:test-events

Conversation

@staabm
Copy link
Copy Markdown
Contributor

@staabm staabm commented Aug 6, 2025

#6288 is currently erroring on Issue5592Test therefore adding more precise assertions arround this area to get a clearer picture and better coverage

requires #6292 to be merged before

Comment on lines +4 to +9
<?php declare(strict_types=1);
$_SERVER['argv'][] = '--do-not-cache-result';
$_SERVER['argv'][] = '--no-configuration';
$_SERVER['argv'][] = '--process-isolation';
$_SERVER['argv'][] = '--debug';
$_SERVER['argv'][] = __DIR__ . '/5592/Issue5592Test.php';
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same test as in tests/end-to-end/regression/5592-process-isolation.phpt, but using --debug.

otherwise no changes

@staabm staabm marked this pull request as ready for review August 6, 2025 07:42
@staabm staabm marked this pull request as draft August 6, 2025 08:31
@staabm staabm marked this pull request as ready for review August 12, 2025 10:59
@staabm staabm marked this pull request as draft August 12, 2025 10:59
@staabm staabm marked this pull request as ready for review August 12, 2025 11:11
@sebastianbergmann sebastianbergmann added the type/tests Tests! Yes, PHPUnit, too, needs tests. label Aug 12, 2025
@sebastianbergmann
Copy link
Copy Markdown
Owner

Cherry-picked into 11.5 and merged to 12.3 and main from there.

@staabm staabm deleted the test-events branch August 12, 2025 12:29
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

type/tests Tests! Yes, PHPUnit, too, needs tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants