From a9357a8311b2a3a9a114ec8400a9878d5f1f8345 Mon Sep 17 00:00:00 2001 From: adev Date: Tue, 9 May 2017 23:19:24 +0200 Subject: [PATCH] Remove the created entry to avoid side effects on other tests --- .../Controller/EntryRestControllerTest.php | 26 +++++++++++++------ 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php index 0979ca93..deafd1fa 100644 --- a/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php +++ b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php @@ -373,14 +373,24 @@ class EntryRestControllerTest extends WallabagApiTestCase ->willThrowException(new \Exception('Test Fetch content fails')); $container->set('wallabag_core.content_proxy', $contentProxy); - $this->client->request('POST', '/api/entries.json', [ - 'url' => 'http://www.example.com/', - ]); - - $this->assertEquals(200, $this->client->getResponse()->getStatusCode()); - $content = json_decode($this->client->getResponse()->getContent(), true); - $this->assertGreaterThan(0, $content['id']); - $this->assertEquals('http://www.example.com/', $content['url']); + try { + $this->client->request('POST', '/api/entries.json', [ + 'url' => 'http://www.example.com/', + ]); + + $this->assertEquals(200, $this->client->getResponse()->getStatusCode()); + $content = json_decode($this->client->getResponse()->getContent(), true); + $this->assertGreaterThan(0, $content['id']); + $this->assertEquals('http://www.example.com/', $content['url']); + } finally { + // Remove the created entry to avoid side effects on other tests + if (isset($content['id'])) { + $em = $this->client->getContainer()->get('doctrine.orm.entity_manager'); + $entry = $em->getReference('WallabagCoreBundle:Entry', $content['id']); + $em->remove($entry); + $em->flush(); + } + } } public function testPostArchivedAndStarredEntry() -- 2.41.0