Skip to content

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