Skip to content

Test events while process isolation #6291

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

staabm
Copy link
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
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
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/tests Tests! Yes, PHPUnit, too, needs tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants