fix tests

ASKozienko · ASKozienko · commit fda9c05b61e0 · 2018-11-05T15:53:09.000+02:00
diff --git a/pkg/enqueue-bundle/Tests/Functional/Client/ProducerTest.php b/pkg/enqueue-bundle/Tests/Functional/Client/ProducerTest.php
@@ -25,9 +25,9 @@ public function setUp()
 
     public function tearDown()
     {
-        parent::tearDown();
-
         static::$container->get(Producer::class)->clearTraces();
+
+        parent::tearDown();
     }
 
     public function testCouldBeGetFromContainerAsService()
diff --git a/pkg/enqueue-bundle/Tests/Functional/Events/AsyncListenerTest.php b/pkg/enqueue-bundle/Tests/Functional/Events/AsyncListenerTest.php
@@ -23,6 +23,8 @@ public function setUp()
         $asyncListener = static::$container->get('enqueue.events.async_listener');
 
         $asyncListener->resetSyncMode();
+        static::$container->get('test_async_subscriber')->calls = [];
+        static::$container->get('test_async_listener')->calls = [];
     }
 
     public function testShouldNotCallRealListenerIfMarkedAsAsync()
@@ -54,8 +56,6 @@ public function testShouldSendMessageToExpectedCommandInsteadOfCallingRealListen
 
         $this->assertCount(1, $traces);
 
-        var_dump($traces);
-
         $this->assertEquals('symfony_events', $traces[0]['command']);
         $this->assertEquals('{"subject":"theSubject","arguments":{"fooArg":"fooVal"}}', $traces[0]['body']);
     }
diff --git a/pkg/enqueue-bundle/Tests/Functional/Events/AsyncSubscriberTest.php b/pkg/enqueue-bundle/Tests/Functional/Events/AsyncSubscriberTest.php
@@ -23,6 +23,8 @@ public function setUp()
         $asyncListener = static::$container->get('enqueue.events.async_listener');
 
         $asyncListener->resetSyncMode();
+        static::$container->get('test_async_subscriber')->calls = [];
+        static::$container->get('test_async_listener')->calls = [];
     }
 
     public function testShouldNotCallRealSubscriberIfMarkedAsAsync()
@@ -35,8 +37,6 @@ public function testShouldNotCallRealSubscriberIfMarkedAsAsync()
         /** @var TestAsyncListener $listener */
         $listener = static::$container->get('test_async_subscriber');
 
-        var_dump($listener->calls);
-
         $this->assertEmpty($listener->calls);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -25,9 +25,9 @@ public function setUp()`
`25`	`25`
`26`	`26`	`public function tearDown()`
`27`	`27`	`{`
`28`		`- parent::tearDown();`
`29`		`-`
`30`	`28`	`static::$container->get(Producer::class)->clearTraces();`
	`29`	`+`
	`30`	`+ parent::tearDown();`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`public function testCouldBeGetFromContainerAsService()`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,8 @@ public function setUp()`
`23`	`23`	`$asyncListener = static::$container->get('enqueue.events.async_listener');`
`24`	`24`
`25`	`25`	`$asyncListener->resetSyncMode();`
	`26`	`+ static::$container->get('test_async_subscriber')->calls = [];`
	`27`	`+ static::$container->get('test_async_listener')->calls = [];`
`26`	`28`	`}`
`27`	`29`
`28`	`30`	`public function testShouldNotCallRealListenerIfMarkedAsAsync()`
`@@ -54,8 +56,6 @@ public function testShouldSendMessageToExpectedCommandInsteadOfCallingRealListen`
`54`	`56`
`55`	`57`	`$this->assertCount(1, $traces);`
`56`	`58`
`57`		`- var_dump($traces);`
`58`		`-`
`59`	`59`	`$this->assertEquals('symfony_events', $traces[0]['command']);`
`60`	`60`	`$this->assertEquals('{"subject":"theSubject","arguments":{"fooArg":"fooVal"}}', $traces[0]['body']);`
`61`	`61`	`}`