Skip to content

Isolate intersection test toggle state - #6255

Merged
ondrejmirtes merged 1 commit into
phpstan:2.3.xfrom
simPod:test/isolate-intersection-toggle-state
Nov 1, 2026
Merged

ondrejmirtes merged 1 commit into
phpstan:2.3.xfrom
simPod:test/isolate-intersection-toggle-state

Conversation

@simPod

@simPod simPod commented Aug 24, 2026 •

Copy link
Copy Markdown
Contributor

Reused ParaTest workers can leave the global bleeding-edge toggle enabled. This changes the array shapes constructed by toggle-sensitive IntersectionType test fixtures and makes their expected trinary results depend on test order.

Construct those fixtures with bleeding edge disabled so they retain their intended unsealed-array semantics without changing the surrounding test state.

Failure in PR #6254: Tests with old PHPUnit (7.4, windows-latest).

Construct legacy ConstantArrayType fixtures with bleeding edge disabled so reused ParaTest workers cannot change the expected acceptance results.
@ondrejmirtes
ondrejmirtes force-pushed the test/isolate-intersection-toggle-state branch from 623157c to 83ccf74 Compare October 31, 2026 13:24
@ondrejmirtes
ondrejmirtes changed the base branch from 2.2.x to 2.3.x October 31, 2026 13:24
@simPod
simPod deleted the test/isolate-intersection-toggle-state branch November 1, 2026 09:43
@ondrejmirtes
ondrejmirtes merged commit 364bd34 into phpstan:2.3.x Nov 1, 2026
724 checks passed
@ondrejmirtes

Copy link
Copy Markdown
Member

Thank you!

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.

2 participants