Merge pull request #8102 from creative-commoners/pulls/4.1/response-assertion

Add test assertion for response instance
This commit is contained in:
Ingo Schommer 2018-06-08 14:08:25 +12:00 committed by GitHub
commit 48304fd6f0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -31,8 +31,13 @@ class DirectorTest extends SapphireTest
Director::config()->set('alternate_base_url', 'http://www.mysite.com:9090/');
// Ensure redirects enabled on all environments
CanonicalURLMiddleware::singleton()->setEnabledEnvs(true);
$middleware = CanonicalURLMiddleware::singleton()->setEnabledEnvs(true);
$this->expectedRedirect = null;
// Ensure global state doesn't affect this test
$middleware
->setForceSSLDomain(null)
->setForceSSLPatterns([]);
}
protected function getExtraRoutes()
@ -593,6 +598,7 @@ class DirectorTest extends SapphireTest
}, 'http://www.mysite.com:9090/some-url');
// Middleware returns non-exception redirect
$this->assertInstanceOf(HTTPResponse::class, $response);
$this->assertEquals('https://www.mysite.com:9090/some-url', $response->getHeader('Location'));
$this->assertEquals(301, $response->getStatusCode());
}