File tree Expand file tree Collapse file tree 4 files changed +21
-2
lines changed Expand file tree Collapse file tree 4 files changed +21
-2
lines changed Original file line number Diff line number Diff line change @@ -56,7 +56,7 @@ public function getKernel(): KernelInterface
5656 */
5757 public function getProfile (): HttpProfile |false |null
5858 {
59- if (null === $ this ->response || !$ this ->getContainer ()->has ('profiler ' )) {
59+ if (! isset ( $ this ->response ) || !$ this ->getContainer ()->has ('profiler ' )) {
6060 return false ;
6161 }
6262
Original file line number Diff line number Diff line change 1414use Symfony \Bundle \FrameworkBundle \KernelBrowser ;
1515use Symfony \Bundle \FrameworkBundle \Tests \Functional \AbstractWebTestCase ;
1616use Symfony \Component \HttpFoundation \Response ;
17+ use Symfony \Component \HttpKernel \KernelInterface ;
1718
1819class KernelBrowserTest extends AbstractWebTestCase
1920{
@@ -61,6 +62,13 @@ public function testRequestAfterKernelShutdownAndPerformedRequest()
6162 $ client ->request ('GET ' , '/ ' );
6263 }
6364
65+ public function testGetProfileWithoutRequest ()
66+ {
67+ $ browser = new KernelBrowser ($ this ->createMock (KernelInterface::class));
68+
69+ $ this ->assertFalse ($ browser ->getProfile ());
70+ }
71+
6472 private function getKernelMock ()
6573 {
6674 $ mock = $ this ->getMockBuilder ($ this ->getKernelClass ())
Original file line number Diff line number Diff line change @@ -531,7 +531,7 @@ public function reload(): Crawler
531531 */
532532 public function followRedirect (): Crawler
533533 {
534- if (!$ this ->redirect ) {
534+ if (!isset ( $ this ->redirect ) ) {
535535 throw new LogicException ('The request was not redirected. ' );
536536 }
537537
Original file line number Diff line number Diff line change 1515use Symfony \Component \BrowserKit \CookieJar ;
1616use Symfony \Component \BrowserKit \Exception \BadMethodCallException ;
1717use Symfony \Component \BrowserKit \Exception \InvalidArgumentException ;
18+ use Symfony \Component \BrowserKit \Exception \LogicException ;
1819use Symfony \Component \BrowserKit \History ;
1920use Symfony \Component \BrowserKit \Request ;
2021use Symfony \Component \BrowserKit \Response ;
@@ -889,4 +890,14 @@ public function testInternalRequestNull()
889890
890891 $ client ->getInternalRequest ();
891892 }
893+
894+ public function testFollowRedirectWithoutRequest ()
895+ {
896+ $ browser = $ this ->getBrowser ();
897+
898+ $ this ->expectException (LogicException::class);
899+ $ this ->expectExceptionMessage ('The request was not redirected. ' );
900+
901+ $ browser ->followRedirect ();
902+ }
892903}
You can’t perform that action at this time.
0 commit comments