Skip to content

Commit 87634cb

Browse files
committed
Merge pull request #291 from symfony-cmf/php7-patch
Test against PHP7
2 parents 22ace07 + a91da72 commit 87634cb

File tree

3 files changed

+22
-8
lines changed

3 files changed

+22
-8
lines changed

.travis.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ php:
66
- 5.5
77
- 5.6
88
- hhvm
9+
- nightly
910

1011
sudo: false
1112

@@ -31,6 +32,7 @@ matrix:
3132
env: SYMFONY_VERSION=2.7.*
3233
- php: 5.6
3334
env: SYMFONY_VERSION=3.0.*
35+
- php: nightly
3436

3537
before_script:
3638
- composer self-update

Tests/WebTest/RedirectRouteAdminTest.php

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
use Symfony\Cmf\Component\Testing\Functional\BaseTestCase;
1515
use Symfony\Component\DomCrawler\Crawler;
16+
use Symfony\Component\HttpFoundation\Response;
1617

1718
class RedirectRouteAdminTest extends BaseTestCase
1819
{
@@ -33,15 +34,15 @@ public function testRedirectRouteList()
3334
{
3435
$crawler = $this->client->request('GET', '/admin/cmf/routing/redirectroute/list');
3536
$res = $this->client->getResponse();
36-
$this->assertEquals(200, $res->getStatusCode());
37+
$this->assertResponseOk($res);
3738
$this->assertCount(1, $crawler->filter('html:contains("redirect-route-1")'));
3839
}
3940

4041
public function testRedirectRouteEdit()
4142
{
4243
$crawler = $this->client->request('GET', '/admin/cmf/routing/redirectroute/test/routing/redirect-route-1/edit');
4344
$res = $this->client->getResponse();
44-
$this->assertEquals(200, $res->getStatusCode());
45+
$this->assertResponseOk($res);
4546
$this->assertCount(1, $crawler->filter('input[value="redirect-route-1"]'));
4647

4748
$this->assertFrontendLinkPresent($crawler);
@@ -51,14 +52,14 @@ public function testRedirectRouteShow()
5152
{
5253
$crawler = $this->client->request('GET', '/admin/cmf/routing/redirectroute/test/routing/redirect-route-1/show');
5354
$res = $this->client->getResponse();
54-
$this->assertEquals(200, $res->getStatusCode());
55+
$this->assertResponseOk($res);
5556
}
5657

5758
public function testRedirectRouteCreate()
5859
{
5960
$crawler = $this->client->request('GET', '/admin/cmf/routing/redirectroute/create');
6061
$res = $this->client->getResponse();
61-
$this->assertEquals(200, $res->getStatusCode());
62+
$this->assertResponseOk($res);
6263

6364
$this->assertFrontendLinkNotPresent($crawler);
6465

@@ -94,4 +95,9 @@ private function assertFrontendLinkNotPresent(Crawler $crawler)
9495
{
9596
$this->assertCount(0, $crawler->filter('a[class="sonata-admin-frontend-link"]'));
9697
}
98+
99+
private function assertResponseOk(Response $response)
100+
{
101+
$this->assertEquals(200, $response->getStatusCode(), $response->getContent());
102+
}
97103
}

Tests/WebTest/RouteAdminTest.php

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
use Symfony\Cmf\Component\Testing\Functional\BaseTestCase;
1515
use Symfony\Component\DomCrawler\Crawler;
16+
use Symfony\Component\HttpFoundation\Response;
1617

1718
class RouteAdminTest extends BaseTestCase
1819
{
@@ -33,15 +34,15 @@ public function testRouteList()
3334
{
3435
$crawler = $this->client->request('GET', '/admin/cmf/routing/route/list');
3536
$res = $this->client->getResponse();
36-
$this->assertEquals(200, $res->getStatusCode());
37+
$this->assertResponseOk($res);
3738
$this->assertCount(1, $crawler->filter('html:contains("route-1")'));
3839
}
3940

4041
public function testRouteEdit()
4142
{
4243
$crawler = $this->client->request('GET', '/admin/cmf/routing/route/test/routing/route-1/edit');
4344
$res = $this->client->getResponse();
44-
$this->assertEquals(200, $res->getStatusCode());
45+
$this->assertResponseOk($res);
4546
$this->assertCount(1, $crawler->filter('input[value="route-1"]'));
4647

4748
$this->assertFrontendLinkPresent($crawler);
@@ -51,14 +52,14 @@ public function testRouteShow()
5152
{
5253
$crawler = $this->client->request('GET', '/admin/cmf/routing/route/test/routing/route-1/show');
5354
$res = $this->client->getResponse();
54-
$this->assertEquals(200, $res->getStatusCode());
55+
$this->assertResponseOk($res);
5556
}
5657

5758
public function testRouteCreate()
5859
{
5960
$crawler = $this->client->request('GET', '/admin/cmf/routing/route/create');
6061
$res = $this->client->getResponse();
61-
$this->assertEquals(200, $res->getStatusCode());
62+
$this->assertResponseOk($res);
6263

6364
$this->assertFrontendLinkNotPresent($crawler);
6465

@@ -94,4 +95,9 @@ private function assertFrontendLinkNotPresent(Crawler $crawler)
9495
{
9596
$this->assertCount(0, $crawler->filter('a[class="sonata-admin-frontend-link"]'));
9697
}
98+
99+
private function assertResponseOk(Response $response)
100+
{
101+
$this->assertEquals(200, $response->getStatusCode(), $response->getContent());
102+
}
97103
}

0 commit comments

Comments
 (0)