minor symfony#25458 [TwigBridge] Fix workflow test with deps=low (Simperfit)

This PR was merged into the 4.1-dev branch.

Discussion
----------

[TwigBridge] Fix workflow test with deps=low

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT
| Doc PR        | none

In the original PR the tests were not failling but we should not use the new feature with deps=low until it fetch the branch with the new feature. So this fix the test for the WorkflowExtension in TwigBridge.

Commits
-------

e5c6b92 [TwigBridge] Fix workflow test with deps=low

Author: nicolas-grekas

src/Symfony/Bridge/Twig/Tests/Extension/WorkflowExtensionTest.php

@@ -15,6 +15,7 @@
 use Symfony\Bridge\Twig\Extension\WorkflowExtension;
 use Symfony\Component\Workflow\Definition;
 use Symfony\Component\Workflow\Registry;
+use Symfony\Component\Workflow\SupportStrategy\ClassInstanceSupportStrategy;
 use Symfony\Component\Workflow\SupportStrategy\InstanceOfSupportStrategy;
 use Symfony\Component\Workflow\Transition;
 use Symfony\Component\Workflow\Workflow;
@@ -24,25 +25,6 @@ class WorkflowExtensionTest extends TestCase
     private $extension;
 
     protected function setUp()
-    {
-        $places = array('ordered', 'waiting_for_payment', 'processed');
-        $transitions = array(
-            new Transition('t1', 'ordered', 'waiting_for_payment'),
-            new Transition('t2', 'waiting_for_payment', 'processed'),
-        );
-        $definition = new Definition($places, $transitions);
-        $workflow = new Workflow($definition);
-
-        $registry = new Registry();
-        $registry->addWorkflow($workflow, new InstanceOfSupportStrategy(\stdClass::class));
-
-        $this->extension = new WorkflowExtension($registry);
-    }
-
-    /**
-     * @group legacy
-     */
-    protected function setUpLegacyAdd()
     {
         if (!class_exists(Workflow::class)) {
             $this->markTestSkipped('The Workflow component is needed to run tests for this extension.');
@@ -57,25 +39,14 @@ protected function setUpLegacyAdd()
         $workflow = new Workflow($definition);
 
         $registry = new Registry();
-        $registry->add($workflow, new InstanceOfSupportStrategy(\stdClass::class));
-
+        $addWorkflow = method_exists($registry, 'addWorkflow') ? 'addWorkflow' : 'add';
+        $supportStrategy = class_exists(InstanceOfSupportStrategy::class)
+            ? new InstanceOfSupportStrategy(\stdClass::class)
+            : new ClassInstanceSupportStrategy(\stdClass::class);
+        $registry->$addWorkflow($workflow, $supportStrategy);
         $this->extension = new WorkflowExtension($registry);
     }
 
-    /**
-     * @group legacy
-     * @expectedDeprecation Symfony\Component\Workflow\Registry::add is deprecated since Symfony 4.1. Use addWorkflow() instead.
-     */
-    public function testCanTransitionLegacy()
-    {
-        $this->setUpLegacyAdd();
-        $subject = new \stdClass();
-        $subject->marking = array();
-
-        $this->assertTrue($this->extension->canTransition($subject, 't1'));
-        $this->assertFalse($this->extension->canTransition($subject, 't2'));
-    }
-
     public function testCanTransition()
     {
         $subject = new \stdClass();