[TwigBridge] Fix workflow test with deps=low

Amrouche Hamza · Amrouche Hamza · commit df5c849f5ba9 · 2017-12-12T11:01:12.000+01:00
diff --git a/src/Symfony/Bridge/Twig/Tests/Extension/WorkflowExtensionTest.php b/src/Symfony/Bridge/Twig/Tests/Extension/WorkflowExtensionTest.php
@@ -23,26 +23,11 @@ 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
+     * @expectedDeprecation Symfony\Component\Workflow\Registry::add is deprecated since Symfony 4.1. Use addWorkflow() instead.
      */
-    protected function setUpLegacyAdd()
+    protected function setUp()
     {
         if (!class_exists(Workflow::class)) {
             $this->markTestSkipped('The Workflow component is needed to run tests for this extension.');
@@ -66,16 +51,6 @@ protected function setUpLegacyAdd()
      * @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();
@@ -85,6 +60,10 @@ public function testCanTransition()
         $this->assertFalse($this->extension->canTransition($subject, 't2'));
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation Symfony\Component\Workflow\Registry::add is deprecated since Symfony 4.1. Use addWorkflow() instead.
+     */
     public function testGetEnabledTransitions()
     {
         $subject = new \stdClass();
@@ -97,6 +76,10 @@ public function testGetEnabledTransitions()
         $this->assertSame('t1', $transitions[0]->getName());
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation Symfony\Component\Workflow\Registry::add is deprecated since Symfony 4.1. Use addWorkflow() instead.
+     */
     public function testHasMarkedPlace()
     {
         $subject = new \stdClass();
@@ -108,6 +91,10 @@ public function testHasMarkedPlace()
         $this->assertFalse($this->extension->hasMarkedPlace($subject, 'processed'));
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation Symfony\Component\Workflow\Registry::add is deprecated since Symfony 4.1. Use addWorkflow() instead.
+     */
     public function testGetMarkedPlaces()
     {
         $subject = new \stdClass();
