Skip to content

Commit 21bd96d

Browse files
committed
Remove unused local variables in tests
1 parent 2ab250e commit 21bd96d

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

Tests/DefinitionBuilderTest.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class DefinitionBuilderTest extends TestCase
1111
public function testAddPlaceInvalidName()
1212
{
1313
$this->expectException('Symfony\Component\Workflow\Exception\InvalidArgumentException');
14-
$builder = new DefinitionBuilder(['a"', 'b']);
14+
new DefinitionBuilder(['a"', 'b']);
1515
}
1616

1717
public function testSetInitialPlace()

Tests/DefinitionTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public function testAddPlacesInvalidArgument()
2222
{
2323
$this->expectException('Symfony\Component\Workflow\Exception\InvalidArgumentException');
2424
$places = ['a"', 'e"'];
25-
$definition = new Definition($places, []);
25+
new Definition($places, []);
2626
}
2727

2828
public function testSetInitialPlace()
@@ -37,7 +37,7 @@ public function testSetInitialPlaceAndPlaceIsNotDefined()
3737
{
3838
$this->expectException('Symfony\Component\Workflow\Exception\LogicException');
3939
$this->expectExceptionMessage('Place "d" cannot be the initial place as it does not exist.');
40-
$definition = new Definition([], [], 'd');
40+
new Definition([], [], 'd');
4141
}
4242

4343
public function testAddTransition()

Tests/TransitionTest.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ public function testValidateName()
1111
{
1212
$this->expectException('Symfony\Component\Workflow\Exception\InvalidArgumentException');
1313
$this->expectExceptionMessage('The transition "foo.bar" contains invalid characters.');
14-
$transition = new Transition('foo.bar', 'a', 'b');
14+
new Transition('foo.bar', 'a', 'b');
1515
}
1616

1717
public function testConstructor()

Tests/WorkflowTest.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ public function testApplyWithSameNameTransition()
219219
$this->assertFalse($marking->has('b'));
220220
$this->assertFalse($marking->has('c'));
221221

222-
$marking = $workflow->apply($subject, 'a_to_bc');
222+
$workflow->apply($subject, 'a_to_bc');
223223
$marking = $workflow->apply($subject, 'b_to_c');
224224

225225
$this->assertFalse($marking->has('a'));
@@ -291,7 +291,7 @@ public function testApplyWithEventDispatcher()
291291
'workflow.workflow_name.announce.t2',
292292
];
293293

294-
$marking = $workflow->apply($subject, 't1');
294+
$workflow->apply($subject, 't1');
295295

296296
$this->assertSame($eventNameExpected, $eventDispatcher->dispatchedEvents);
297297
}
@@ -330,7 +330,7 @@ public function testApplyDoesNotTriggerExtraGuardWithEventDispatcher()
330330
'workflow.workflow_name.announce',
331331
];
332332

333-
$marking = $workflow->apply($subject, 'a-b');
333+
$workflow->apply($subject, 'a-b');
334334

335335
$this->assertSame($eventNameExpected, $eventDispatcher->dispatchedEvents);
336336
}

0 commit comments

Comments
 (0)