@@ -27,7 +27,7 @@ public function testGetMarkingWithInvalidStoreReturn()
2727 {
2828 $ subject = new \stdClass ();
2929 $ subject ->marking = null ;
30- $ workflow = new Workflow (new Definition (array (), array () ), $ this ->getMockBuilder (MarkingStoreInterface::class)->getMock ());
30+ $ workflow = new Workflow (new Definition ([], [] ), $ this ->getMockBuilder (MarkingStoreInterface::class)->getMock ());
3131
3232 $ workflow ->getMarking ($ subject );
3333 }
@@ -40,7 +40,7 @@ public function testGetMarkingWithEmptyDefinition()
4040 {
4141 $ subject = new \stdClass ();
4242 $ subject ->marking = null ;
43- $ workflow = new Workflow (new Definition (array (), array () ), new MultipleStateMarkingStore ());
43+ $ workflow = new Workflow (new Definition ([], [] ), new MultipleStateMarkingStore ());
4444
4545 $ workflow ->getMarking ($ subject );
4646 }
@@ -52,8 +52,8 @@ public function testGetMarkingWithEmptyDefinition()
5252 public function testGetMarkingWithImpossiblePlace ()
5353 {
5454 $ subject = new \stdClass ();
55- $ subject ->marking = array ( 'nope ' => 1 ) ;
56- $ workflow = new Workflow (new Definition (array (), array () ), new MultipleStateMarkingStore ());
55+ $ subject ->marking = [ 'nope ' => 1 ] ;
56+ $ workflow = new Workflow (new Definition ([], [] ), new MultipleStateMarkingStore ());
5757
5858 $ workflow ->getMarking ($ subject );
5959 }
@@ -69,15 +69,15 @@ public function testGetMarkingWithEmptyInitialMarking()
6969
7070 $ this ->assertInstanceOf (Marking::class, $ marking );
7171 $ this ->assertTrue ($ marking ->has ('a ' ));
72- $ this ->assertSame (array ( 'a ' => 1 ) , $ subject ->marking );
72+ $ this ->assertSame ([ 'a ' => 1 ] , $ subject ->marking );
7373 }
7474
7575 public function testGetMarkingWithExistingMarking ()
7676 {
7777 $ definition = $ this ->createComplexWorkflowDefinition ();
7878 $ subject = new \stdClass ();
7979 $ subject ->marking = null ;
80- $ subject ->marking = array ( 'b ' => 1 , 'c ' => 1 ) ;
80+ $ subject ->marking = [ 'b ' => 1 , 'c ' => 1 ] ;
8181 $ workflow = new Workflow ($ definition , new MultipleStateMarkingStore ());
8282
8383 $ marking = $ workflow ->getMarking ($ subject );
@@ -107,19 +107,19 @@ public function testCan()
107107 $ this ->assertTrue ($ workflow ->can ($ subject , 't1 ' ));
108108 $ this ->assertFalse ($ workflow ->can ($ subject , 't2 ' ));
109109
110- $ subject ->marking = array ( 'b ' => 1 ) ;
110+ $ subject ->marking = [ 'b ' => 1 ] ;
111111
112112 $ this ->assertFalse ($ workflow ->can ($ subject , 't1 ' ));
113113 // In a workflow net, all "from" places should contain a token to enable
114114 // the transition.
115115 $ this ->assertFalse ($ workflow ->can ($ subject , 't2 ' ));
116116
117- $ subject ->marking = array ( 'b ' => 1 , 'c ' => 1 ) ;
117+ $ subject ->marking = [ 'b ' => 1 , 'c ' => 1 ] ;
118118
119119 $ this ->assertFalse ($ workflow ->can ($ subject , 't1 ' ));
120120 $ this ->assertTrue ($ workflow ->can ($ subject , 't2 ' ));
121121
122- $ subject ->marking = array ( 'f ' => 1 ) ;
122+ $ subject ->marking = [ 'f ' => 1 ] ;
123123
124124 $ this ->assertFalse ($ workflow ->can ($ subject , 't5 ' ));
125125 $ this ->assertTrue ($ workflow ->can ($ subject , 't6 ' ));
@@ -145,7 +145,7 @@ public function testCanDoesNotTriggerGuardEventsForNotEnabledTransitions()
145145 $ subject = new \stdClass ();
146146 $ subject ->marking = null ;
147147
148- $ dispatchedEvents = array () ;
148+ $ dispatchedEvents = [] ;
149149 $ eventDispatcher = new EventDispatcher ();
150150
151151 $ workflow = new Workflow ($ definition , new MultipleStateMarkingStore (), $ eventDispatcher , 'workflow_name ' );
@@ -161,7 +161,7 @@ public function testCanDoesNotTriggerGuardEventsForNotEnabledTransitions()
161161
162162 $ workflow ->can ($ subject , 't3 ' );
163163
164- $ this ->assertSame (array ( 'workflow_name.guard.t3 ' ) , $ dispatchedEvents );
164+ $ this ->assertSame ([ 'workflow_name.guard.t3 ' ] , $ dispatchedEvents );
165165 }
166166
167167 public function testCanWithSameNameTransition ()
@@ -175,7 +175,7 @@ public function testCanWithSameNameTransition()
175175 $ this ->assertFalse ($ workflow ->can ($ subject , 'b_to_c ' ));
176176 $ this ->assertFalse ($ workflow ->can ($ subject , 'to_a ' ));
177177
178- $ subject ->marking = array ( 'b ' => 1 ) ;
178+ $ subject ->marking = [ 'b ' => 1 ] ;
179179 $ this ->assertFalse ($ workflow ->can ($ subject , 'a_to_bc ' ));
180180 $ this ->assertTrue ($ workflow ->can ($ subject , 'b_to_c ' ));
181181 $ this ->assertTrue ($ workflow ->can ($ subject , 'to_a ' ));
@@ -205,17 +205,17 @@ public function testBuildTransitionBlockerList()
205205 $ this ->assertTrue ($ workflow ->buildTransitionBlockerList ($ subject , 't1 ' )->isEmpty ());
206206 $ this ->assertFalse ($ workflow ->buildTransitionBlockerList ($ subject , 't2 ' )->isEmpty ());
207207
208- $ subject ->marking = array ( 'b ' => 1 ) ;
208+ $ subject ->marking = [ 'b ' => 1 ] ;
209209
210210 $ this ->assertFalse ($ workflow ->buildTransitionBlockerList ($ subject , 't1 ' )->isEmpty ());
211211 $ this ->assertFalse ($ workflow ->buildTransitionBlockerList ($ subject , 't2 ' )->isEmpty ());
212212
213- $ subject ->marking = array ( 'b ' => 1 , 'c ' => 1 ) ;
213+ $ subject ->marking = [ 'b ' => 1 , 'c ' => 1 ] ;
214214
215215 $ this ->assertFalse ($ workflow ->buildTransitionBlockerList ($ subject , 't1 ' )->isEmpty ());
216216 $ this ->assertTrue ($ workflow ->buildTransitionBlockerList ($ subject , 't2 ' )->isEmpty ());
217217
218- $ subject ->marking = array ( 'f ' => 1 ) ;
218+ $ subject ->marking = [ 'f ' => 1 ] ;
219219
220220 $ this ->assertFalse ($ workflow ->buildTransitionBlockerList ($ subject , 't5 ' )->isEmpty ());
221221 $ this ->assertTrue ($ workflow ->buildTransitionBlockerList ($ subject , 't6 ' )->isEmpty ());
@@ -354,10 +354,10 @@ public function testApplyWithSameNameTransition()
354354 public function testApplyWithSameNameTransition2 ()
355355 {
356356 $ subject = new \stdClass ();
357- $ subject ->marking = array ( 'a ' => 1 , 'b ' => 1 ) ;
357+ $ subject ->marking = [ 'a ' => 1 , 'b ' => 1 ] ;
358358
359359 $ places = range ('a ' , 'd ' );
360- $ transitions = array () ;
360+ $ transitions = [] ;
361361 $ transitions [] = new Transition ('t ' , 'a ' , 'c ' );
362362 $ transitions [] = new Transition ('t ' , 'b ' , 'd ' );
363363 $ definition = new Definition ($ places , $ transitions );
@@ -374,10 +374,10 @@ public function testApplyWithSameNameTransition2()
374374 public function testApplyWithSameNameTransition3 ()
375375 {
376376 $ subject = new \stdClass ();
377- $ subject ->marking = array ( 'a ' => 1 ) ;
377+ $ subject ->marking = [ 'a ' => 1 ] ;
378378
379379 $ places = range ('a ' , 'd ' );
380- $ transitions = array () ;
380+ $ transitions = [] ;
381381 $ transitions [] = new Transition ('t ' , 'a ' , 'b ' );
382382 $ transitions [] = new Transition ('t ' , 'b ' , 'c ' );
383383 $ transitions [] = new Transition ('t ' , 'c ' , 'd ' );
@@ -398,7 +398,7 @@ public function testApplyWithEventDispatcher()
398398 $ eventDispatcher = new EventDispatcherMock ();
399399 $ workflow = new Workflow ($ definition , new MultipleStateMarkingStore (), $ eventDispatcher , 'workflow_name ' );
400400
401- $ eventNameExpected = array (
401+ $ eventNameExpected = [
402402 'workflow.entered ' ,
403403 'workflow.workflow_name.entered ' ,
404404 'workflow.guard ' ,
@@ -428,7 +428,7 @@ public function testApplyWithEventDispatcher()
428428 'workflow.workflow_name.guard ' ,
429429 'workflow.workflow_name.guard.t2 ' ,
430430 'workflow.workflow_name.announce.t2 ' ,
431- ) ;
431+ ] ;
432432
433433 $ marking = $ workflow ->apply ($ subject , 't1 ' );
434434
@@ -448,14 +448,14 @@ public function testEventName()
448448 $ this ->assertEquals ($ name , $ event ->getWorkflowName ());
449449 };
450450
451- $ eventNames = array (
451+ $ eventNames = [
452452 'workflow.guard ' ,
453453 'workflow.leave ' ,
454454 'workflow.transition ' ,
455455 'workflow.enter ' ,
456456 'workflow.entered ' ,
457457 'workflow.announce ' ,
458- ) ;
458+ ] ;
459459
460460 foreach ($ eventNames as $ eventName ) {
461461 $ dispatcher ->addListener ($ eventName , $ assertWorkflowName );
@@ -466,20 +466,20 @@ public function testEventName()
466466
467467 public function testMarkingStateOnApplyWithEventDispatcher ()
468468 {
469- $ definition = new Definition (range ('a ' , 'f ' ), array ( new Transition ('t ' , range ('a ' , 'c ' ), range ('d ' , 'f ' ))) );
469+ $ definition = new Definition (range ('a ' , 'f ' ), [ new Transition ('t ' , range ('a ' , 'c ' ), range ('d ' , 'f ' ))] );
470470
471471 $ subject = new \stdClass ();
472- $ subject ->marking = array ( 'a ' => 1 , 'b ' => 1 , 'c ' => 1 ) ;
472+ $ subject ->marking = [ 'a ' => 1 , 'b ' => 1 , 'c ' => 1 ] ;
473473
474474 $ dispatcher = new EventDispatcher ();
475475
476476 $ workflow = new Workflow ($ definition , new MultipleStateMarkingStore (), $ dispatcher , 'test ' );
477477
478478 $ assertInitialState = function (Event $ event ) {
479- $ this ->assertEquals (new Marking (array ( 'a ' => 1 , 'b ' => 1 , 'c ' => 1 ) ), $ event ->getMarking ());
479+ $ this ->assertEquals (new Marking ([ 'a ' => 1 , 'b ' => 1 , 'c ' => 1 ] ), $ event ->getMarking ());
480480 };
481481 $ assertTransitionState = function (Event $ event ) {
482- $ this ->assertEquals (new Marking (array () ), $ event ->getMarking ());
482+ $ this ->assertEquals (new Marking ([] ), $ event ->getMarking ());
483483 };
484484
485485 $ dispatcher ->addListener ('workflow.leave ' , $ assertInitialState );
@@ -512,13 +512,13 @@ public function testGetEnabledTransitions()
512512
513513 $ this ->assertEmpty ($ workflow ->getEnabledTransitions ($ subject ));
514514
515- $ subject ->marking = array ( 'd ' => 1 ) ;
515+ $ subject ->marking = [ 'd ' => 1 ] ;
516516 $ transitions = $ workflow ->getEnabledTransitions ($ subject );
517517 $ this ->assertCount (2 , $ transitions );
518518 $ this ->assertSame ('t3 ' , $ transitions [0 ]->getName ());
519519 $ this ->assertSame ('t4 ' , $ transitions [1 ]->getName ());
520520
521- $ subject ->marking = array ( 'c ' => 1 , 'e ' => 1 ) ;
521+ $ subject ->marking = [ 'c ' => 1 , 'e ' => 1 ] ;
522522 $ transitions = $ workflow ->getEnabledTransitions ($ subject );
523523 $ this ->assertCount (1 , $ transitions );
524524 $ this ->assertSame ('t5 ' , $ transitions [0 ]->getName ());
@@ -535,7 +535,7 @@ public function testGetEnabledTransitionsWithSameNameTransition()
535535 $ this ->assertCount (1 , $ transitions );
536536 $ this ->assertSame ('a_to_bc ' , $ transitions [0 ]->getName ());
537537
538- $ subject ->marking = array ( 'b ' => 1 , 'c ' => 1 ) ;
538+ $ subject ->marking = [ 'b ' => 1 , 'c ' => 1 ] ;
539539 $ transitions = $ workflow ->getEnabledTransitions ($ subject );
540540 $ this ->assertCount (3 , $ transitions );
541541 $ this ->assertSame ('b_to_c ' , $ transitions [0 ]->getName ());
@@ -546,7 +546,7 @@ public function testGetEnabledTransitionsWithSameNameTransition()
546546
547547class EventDispatcherMock implements \Symfony \Component \EventDispatcher \EventDispatcherInterface
548548{
549- public $ dispatchedEvents = array () ;
549+ public $ dispatchedEvents = [] ;
550550
551551 public function dispatch ($ eventName , \Symfony \Component \EventDispatcher \Event $ event = null )
552552 {
0 commit comments