Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

Commit 9767102

Browse files
committed
Better debug functional tests
1 parent 7f36a0f commit 9767102

File tree

7 files changed

+22
-24
lines changed

7 files changed

+22
-24
lines changed

Tests/WebTest/Admin/Extension/MenuNodeReferrersExtensionTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public function testEdit()
2727
{
2828
$crawler = $this->client->request('GET', '/admin/cmf/menu-test/content/test/content-1/edit');
2929
$res = $this->client->getResponse();
30-
$this->assertEquals(200, $res->getStatusCode());
30+
$this->assertResponseSuccess($res);
3131

3232
$button = $crawler->selectButton('Update');
3333
$form = $button->form();

Tests/WebTest/Admin/MenuAdminTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,31 +28,31 @@ public function testMenuList()
2828
{
2929
$crawler = $this->client->request('GET', '/admin/cmf/menu/menu/list');
3030
$res = $this->client->getResponse();
31-
$this->assertEquals(200, $res->getStatusCode(), $res->getContent());
31+
$this->assertResponseSuccess($res);
3232
$this->assertCount(1, $crawler->filter('html:contains("test-menu")'), $res->getContent());
3333
}
3434

3535
public function testMenuEdit()
3636
{
3737
$crawler = $this->client->request('GET', '/admin/cmf/menu/menu/test/menus/test-menu/edit');
3838
$res = $this->client->getResponse();
39-
$this->assertEquals(200, $res->getStatusCode(), $res->getContent());
39+
$this->assertResponseSuccess($res);
4040
$this->assertCount(1, $crawler->filter('input[value="test-menu"]'), $res->getContent());
4141
}
4242

4343
public function testMenuShow()
4444
{
4545
$crawler = $this->client->request('GET', '/admin/cmf/menu/menu/test/menus/test-menu/show');
4646
$res = $this->client->getResponse();
47-
$this->assertEquals(200, $res->getStatusCode(), $res->getContent());
47+
$this->assertResponseSuccess($res);
4848
$this->assertCount(2, $crawler->filter('td:contains("test-menu")'), $res->getContent());
4949
}
5050

5151
public function testMenuCreate()
5252
{
5353
$crawler = $this->client->request('GET', '/admin/cmf/menu/menu/create');
5454
$res = $this->client->getResponse();
55-
$this->assertEquals(200, $res->getStatusCode(), $res->getContent());
55+
$this->assertResponseSuccess($res);
5656

5757
$button = $crawler->selectButton('Create');
5858
$form = $button->form();
@@ -74,7 +74,7 @@ public function testMenuDelete()
7474
{
7575
$crawler = $this->client->request('GET', '/admin/cmf/menu/menu/test/menus/test-menu/delete');
7676
$res = $this->client->getResponse();
77-
$this->assertEquals(200, $res->getStatusCode());
77+
$this->assertResponseSuccess($res);
7878

7979
$button = $crawler->selectButton('Yes, delete');
8080
$form = $button->form();

Tests/WebTest/Admin/MenuNodeAdminTest.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,14 @@ public function setUp()
2626
public function testEdit()
2727
{
2828
$this->client->request('GET', '/admin/cmf/menu/menunode/test/menus/test-menu/item-1/edit');
29-
$res = $this->client->getResponse();
30-
$this->assertEquals(200, $res->getStatusCode());
29+
30+
$this->assertResponseSuccess($this->client->getResponse());
3131
}
3232

3333
public function testDelete()
3434
{
3535
$crawler = $this->client->request('GET', '/admin/cmf/menu/menunode/test/menus/test-menu/item-2/delete');
36-
$res = $this->client->getResponse();
37-
$this->assertEquals(200, $res->getStatusCode());
36+
$this->assertResponseSuccess($this->client->getResponse());
3837

3938
$button = $crawler->selectButton('Yes, delete');
4039
$form = $button->form();

Tests/WebTest/Render/TwigTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,8 @@ public function testTwig()
2727
{
2828
$client = $this->createClient();
2929
$crawler = $client->request('GET', '/render-test');
30-
$res = $client->getResponse();
3130

32-
$this->assertEquals(200, $res->getStatusCode());
31+
$this->assertResponseSuccess($client->getResponse());
3332
$this->assertMenuHasItems($crawler->filter('#content ul')->eq(0), array(
3433
'This node has a URI',
3534
'@todo this node should have content',

Tests/WebTest/Voter/RequestContentIdentityVoterTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ public function testRequestContentIdentityVoterNoOp()
1919
// the URL of the content is the same as the URL for the menu item anyway
2020
// so it works by default
2121
$crawler = $this->client->request('GET', '/contents/content-1');
22-
$res = $this->client->getResponse();
22+
23+
$this->assertResponseSuccess($this->client->getResponse());
2324
$this->assertCurrentItem($crawler, 'Request Content Identity Voter');
24-
$this->assertEquals(200, $res->getStatusCode());
2525
}
2626

2727
public function testRequestContentIdentityVoter()
@@ -30,8 +30,8 @@ public function testRequestContentIdentityVoter()
3030
// the URL is different from that of the content, so if the menu item
3131
// is highlighted, it is because the voter is working.
3232
$crawler = $this->client->request('GET', '/cmi/request_content_identity');
33-
$res = $this->client->getResponse();
33+
34+
$this->assertResponseSuccess($this->client->getResponse());
3435
$this->assertCurrentItem($crawler, 'Request Content Identity Voter');
35-
$this->assertEquals(200, $res->getStatusCode());
3636
}
3737
}

Tests/WebTest/Voter/RequestParentContentIdentityVoterTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,17 +19,17 @@ public function testRequestContentParentIdentityNoOp()
1919
// to the "Request Content PArent Identity" menu item and so DOES NOT invoke
2020
// the voter.
2121
$crawler = $this->client->request('GET', '/blog');
22+
23+
$this->assertResponseSuccess($this->client->getResponse());
2224
$this->assertCurrentItem($crawler, 'Request Parent Content Identity Voter');
23-
$res = $this->client->getResponse();
24-
$this->assertEquals(200, $res->getStatusCode());
2525
}
2626

2727
public function testRequestContentParentIdentity()
2828
{
2929
// this test shows an post whose parent is the blog content referenced in the menu item
3030
$crawler = $this->client->request('GET', '/blog/my-post');
31-
$res = $this->client->getResponse();
31+
32+
$this->assertResponseSuccess($this->client->getResponse());
3233
$this->assertCurrentItem($crawler, 'Request Parent Content Identity Voter');
33-
$this->assertEquals(200, $res->getStatusCode());
3434
}
3535
}

Tests/WebTest/Voter/UriPrefixVoterTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,18 +19,18 @@ public function testUriPrefixArticlesHomepage()
1919
// to the "URI Prefix Voter" menu item and so DOES NOT invoke
2020
// the voter.
2121
$crawler = $this->client->request('GET', '/articles');
22-
$res = $this->client->getResponse();
22+
23+
$this->assertResponseSuccess($this->client->getResponse());
2324
$this->assertCurrentItem($crawler, 'URI Prefix Voter');
24-
$this->assertEquals(200, $res->getStatusCode());
2525
}
2626

2727
public function testUriPrefixArticle()
2828
{
2929
// this test shows an article which contains the prefix in the "/articles" route
3030
// as currentUriPrefix, and so the Voter IS used and the item should be selected.
3131
$crawler = $this->client->request('GET', '/articles/some-category/article-1');
32-
$res = $this->client->getResponse();
32+
33+
$this->assertResponseSuccess($this->client->getResponse());
3334
$this->assertCurrentItem($crawler, 'URI Prefix Voter');
34-
$this->assertEquals(200, $res->getStatusCode());
3535
}
3636
}

0 commit comments

Comments
 (0)