Skip to content

[Tests] Expect twig composer-based set in SetManagerTest - #8279

Merged
TomasVotruba merged 1 commit into
mainfrom
fix-set-manager-twig-composer-based
Aug 3, 2026
Merged

[Tests] Expect twig composer-based set in SetManagerTest#8279
TomasVotruba merged 1 commit into
mainfrom
fix-set-manager-twig-composer-based

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

rector-symfony's TwigSetProvider now provides a twig/composer-based.php set constrained to twig/twig >=1.12, on top of the 7 version-pinned sets. Every twig fixture project in SetManagerTest installs twig >=1.12, so the composer-based set is matched for all of them.

-        $this->assertCount(7, $twigComposerTriggeredSet);
+        $this->assertCount(8, $twigComposerTriggeredSet);
-        yield [__DIR__ . '/Fixture/project-twig-20', [realpath(TwigSetList::TWIG_20)]];
+        yield [
+            __DIR__ . '/Fixture/project-twig-20',
+            [realpath(TwigSetList::COMPOSER_BASED), realpath(TwigSetList::TWIG_20)],
+        ];

Test-only change; vendor/bin/phpunit is green with it.

rector-symfony added a `twig/composer-based.php` ComposerTriggeredSet
constrained to `twig/twig >=1.12`, so it is now matched by every
twig fixture project.
@TomasVotruba
TomasVotruba merged commit 4e11a40 into main Aug 3, 2026
64 checks passed
@TomasVotruba
TomasVotruba deleted the fix-set-manager-twig-composer-based branch August 3, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant