8000 minor #22271 [Workflow] fix risky tests (xabbuh) · symfony/symfony@ada8a29 · GitHub
[go: up one dir, main page]

Skip to content

Commit ada8a29

Browse files
committed
minor #22271 [Workflow] fix risky tests (xabbuh)
This PR was merged into the 3.2 branch. Discussion ---------- [Workflow] fix risky tests | Q | A | ------------- | --- | Branch? | 3.2 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Commits ------- 4e661e7 [Workflow] fix risky tests
2 parents ccbbff2 + 4e661e7 commit ada8a29

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

src/Symfony/Component/Workflow/Tests/Validator/StateMachineValidatorTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,9 @@ public function testValid()
9494

9595
(new StateMachineValidator())->validate($definition, 'foo');
9696

97+
// the test simply ensures that the validation does not fail (i.e. it does not throw any exceptions)
98+
$this->addToAssertionCount(1);
99+
97100
// The graph looks like:
98101
//
99102
// +----+ +----+ +---+

src/Symfony/Component/Workflow/Tests/Validator/WorkflowValidatorTest.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,9 @@ public function testSinglePlaceWorkflowValidatorAndSimpleWorkflow()
2828
$definition = $this->createSimpleWorkflowDefinition();
2929

3030
(new WorkflowValidator(true))->validate($definition, 'foo');
31+
32+
// the test simply ensures that the validation does not fail (i.e. it does not throw any exceptions)
33+
$this->addToAssertionCount(1);
3134
}
3235

3336
/**
@@ -60,5 +63,8 @@ public function testSameTransitionNameButNotSamePlace()
6063
$definition = new Definition($places, $transitions);
6164

6265
(new WorkflowValidator())->validate($definition, 'foo');
66+
67+
// the test simply ensures that the validation does not fail (i.e. it does not throw any exceptions)
68+
$this->addToAssertionCount(1);
6369
}
6470
}

0 commit comments

Comments
 (0)
0