minor #53166 [Workflow] Fix test (OskarStark)

fabpot · fabpot · commit 100e5bc11a0a · 2023-12-21T10:58:56.000+01:00
This PR was merged into the 5.4 branch. Discussion ---------- [Workflow] Fix test | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | Follows #52402 (comment) | License | MIT Commits ------- 02fa2ec [Workflow] Fix test
diff --git a/src/Symfony/Component/Workflow/Tests/Validator/StateMachineValidatorTest.php b/src/Symfony/Component/Workflow/Tests/Validator/StateMachineValidatorTest.php
@@ -116,27 +116,13 @@ public function testValid()
 
     public function testWithTooManyInitialPlaces()
     {
-        $this->expectException(InvalidDefinitionException::class);
-        $this->expectExceptionMessage('The state machine "foo" cannot store many places. But the definition has 2 initial places. Only one is supported.');
         $places = range('a', 'c');
         $transitions = [];
         $definition = new Definition($places, $transitions, ['a', 'b']);
 
-        (new StateMachineValidator())->validate($definition, 'foo');
-
-        // the test ensures that the validation does not fail (i.e. it does not throw any exceptions)
-        $this->addToAssertionCount(1);
+        $this->expectException(InvalidDefinitionException::class);
+        $this->expectExceptionMessage('The state machine "foo" cannot store many places. But the definition has 2 initial places. Only one is supported.');
 
-        // The graph looks like:
-        //
-        // +----+     +----+     +---+
-        // | a  | --> | t1 | --> | b |
-        // +----+     +----+     +---+
-        //   |
-        //   |
-        //   v
-        // +----+     +----+
-        // | t2 | --> | c  |
-        // +----+     +----+
+        (new StateMachineValidator())->validate($definition, 'foo');
     }
 }
