Merge branch '5.2' into 5.x

fabpot · fabpot · commit 946c3617f570 · 2021-01-11T10:52:33.000+01:00
* 5.2:
  Use ::class keyword when possible
diff --git a/Tests/AbstractBrowserTest.php b/Tests/AbstractBrowserTest.php
@@ -88,7 +88,7 @@ public function testGetResponse()
         $client->request('GET', 'http://example.com/');
 
         $this->assertSame('foo', $client->getResponse()->getContent(), '->getCrawler() returns the Response of the last request');
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Response', $client->getResponse(), '->getCrawler() returns the Response of the last request');
+        $this->assertInstanceOf(Response::class, $client->getResponse(), '->getCrawler() returns the Response of the last request');
     }
 
     public function testGetResponseNull()
@@ -296,7 +296,7 @@ public function testClickLinkNotFound()
             $client->clickLink('foo');
             $this->fail('->clickLink() throws a \InvalidArgumentException if the link could not be found');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('InvalidArgumentException', $e, '->clickLink() throws a \InvalidArgumentException if the link could not be found');
+            $this->assertInstanceOf(\InvalidArgumentException::class, $e, '->clickLink() throws a \InvalidArgumentException if the link could not be found');
         }
     }
 
@@ -355,7 +355,7 @@ public function testSubmitFormNotFound()
             ], 'POST');
             $this->fail('->submitForm() throws a \InvalidArgumentException if the form could not be found');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('InvalidArgumentException', $e, '->submitForm() throws a \InvalidArgumentException if the form could not be found');
+            $this->assertInstanceOf(\InvalidArgumentException::class, $e, '->submitForm() throws a \InvalidArgumentException if the form could not be found');
         }
     }
 
@@ -406,7 +406,7 @@ public function testFollowRedirect()
             $client->followRedirect();
             $this->fail('->followRedirect() throws a \LogicException if the request was not redirected');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('LogicException', $e, '->followRedirect() throws a \LogicException if the request was not redirected');
+            $this->assertInstanceOf(\LogicException::class, $e, '->followRedirect() throws a \LogicException if the request was not redirected');
         }
 
         $client->setNextResponse(new Response('', 302, ['Location' => 'http://www.example.com/redirected']));
@@ -436,7 +436,7 @@ public function testFollowRedirect()
             $client->followRedirect();
             $this->fail('->followRedirect() throws a \LogicException if the request did not respond with 30x HTTP Code');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('LogicException', $e, '->followRedirect() throws a \LogicException if the request did not respond with 30x HTTP Code');
+            $this->assertInstanceOf(\LogicException::class, $e, '->followRedirect() throws a \LogicException if the request did not respond with 30x HTTP Code');
         }
     }
 
@@ -466,7 +466,7 @@ public function testFollowRedirectWithMaxRedirects()
             $client->followRedirect();
             $this->fail('->followRedirect() throws a \LogicException if the request was redirected and limit of redirections was reached');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('LogicException', $e, '->followRedirect() throws a \LogicException if the request was redirected and limit of redirections was reached');
+            $this->assertInstanceOf(\LogicException::class, $e, '->followRedirect() throws a \LogicException if the request was redirected and limit of redirections was reached');
         }
 
         $client->setNextResponse(new Response('', 302, ['Location' => 'http://www.example.com/redirected']));
@@ -749,7 +749,7 @@ public function testInsulatedRequests()
             $client->request('GET', 'http://www.example.com/foo/foobar');
             $this->fail('->request() throws a \RuntimeException if the script has an error');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('RuntimeException', $e, '->request() throws a \RuntimeException if the script has an error');
+            $this->assertInstanceOf(\RuntimeException::class, $e, '->request() throws a \RuntimeException if the script has an error');
         }
     }
 
@@ -837,7 +837,7 @@ public function testInternalRequest()
             'NEW_SERVER_KEY' => 'new-server-key-value',
         ]);
 
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Request', $client->getInternalRequest());
+        $this->assertInstanceOf(\Symfony\Component\BrowserKit\Request::class, $client->getInternalRequest());
     }
 
     public function testInternalRequestNull()
diff --git a/Tests/CookieJarTest.php b/Tests/CookieJarTest.php
@@ -77,8 +77,8 @@ public function testUpdateFromSetCookie()
         $cookieJar->set(new Cookie('bar', 'bar'));
         $cookieJar->updateFromSetCookie($setCookies);
 
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Cookie', $cookieJar->get('foo'));
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Cookie', $cookieJar->get('bar'));
+        $this->assertInstanceOf(Cookie::class, $cookieJar->get('foo'));
+        $this->assertInstanceOf(Cookie::class, $cookieJar->get('bar'));
         $this->assertEquals('foo', $cookieJar->get('foo')->getValue(), '->updateFromSetCookie() updates cookies from a Set-Cookie header');
         $this->assertEquals('bar', $cookieJar->get('bar')->getValue(), '->updateFromSetCookie() keeps existing cookies');
     }
@@ -103,9 +103,9 @@ public function testUpdateFromSetCookieWithMultipleCookies()
         $barCookie = $cookieJar->get('bar', '/', '.blog.symfony.com');
         $phpCookie = $cookieJar->get('PHPSESSID');
 
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Cookie', $fooCookie);
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Cookie', $barCookie);
-        $this->assertInstanceOf('Symfony\Component\BrowserKit\Cookie', $phpCookie);
+        $this->assertInstanceOf(Cookie::class, $fooCookie);
+        $this->assertInstanceOf(Cookie::class, $barCookie);
+        $this->assertInstanceOf(Cookie::class, $phpCookie);
         $this->assertEquals('foo', $fooCookie->getValue());
         $this->assertEquals('bar', $barCookie->getValue());
         $this->assertEquals('id', $phpCookie->getValue());
diff --git a/Tests/CookieTest.php b/Tests/CookieTest.php
@@ -104,7 +104,7 @@ public function testFromStringWithUrl()
 
     public function testFromStringThrowsAnExceptionIfCookieIsNotValid()
     {
-        $this->expectException('InvalidArgumentException');
+        $this->expectException(\InvalidArgumentException::class);
         Cookie::fromString('foo');
     }
 
@@ -117,7 +117,7 @@ public function testFromStringIgnoresInvalidExpiresDate()
 
     public function testFromStringThrowsAnExceptionIfUrlIsNotValid()
     {
-        $this->expectException('InvalidArgumentException');
+        $this->expectException(\InvalidArgumentException::class);
         Cookie::fromString('foo=bar', 'foobar');
     }
 
@@ -200,7 +200,7 @@ public function testIsExpired()
 
     public function testConstructException()
     {
-        $this->expectException('UnexpectedValueException');
+        $this->expectException(\UnexpectedValueException::class);
         $this->expectExceptionMessage('The cookie expiration time "string" is not valid.');
         new Cookie('foo', 'bar', 'string');
     }
diff --git a/Tests/HistoryTest.php b/Tests/HistoryTest.php
@@ -55,7 +55,7 @@ public function testCurrent()
             $history->current();
             $this->fail('->current() throws a \LogicException if the history is empty');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('LogicException', $e, '->current() throws a \LogicException if the history is empty');
+            $this->assertInstanceOf(\LogicException::class, $e, '->current() throws a \LogicException if the history is empty');
         }
 
         $history->add(new Request('http://www.example.com/', 'get'));
@@ -72,7 +72,7 @@ public function testBack()
             $history->back();
             $this->fail('->back() throws a \LogicException if the history is already on the first page');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('LogicException', $e, '->current() throws a \LogicException if the history is already on the first page');
+            $this->assertInstanceOf(\LogicException::class, $e, '->current() throws a \LogicException if the history is already on the first page');
         }
 
         $history->add(new Request('http://www.example1.com/', 'get'));
@@ -91,7 +91,7 @@ public function testForward()
             $history->forward();
             $this->fail('->forward() throws a \LogicException if the history is already on the last page');
         } catch (\Exception $e) {
-            $this->assertInstanceOf('LogicException', $e, '->forward() throws a \LogicException if the history is already on the last page');
+            $this->assertInstanceOf(\LogicException::class, $e, '->forward() throws a \LogicException if the history is already on the last page');
         }
 
         $history->back();
diff --git a/Tests/TestClient.php b/Tests/TestClient.php
@@ -43,7 +43,7 @@ protected function doRequest($request): Response
 
     protected function getScript($request)
     {
-        $r = new \ReflectionClass('Symfony\Component\BrowserKit\Response');
+        $r = new \ReflectionClass(Response::class);
         $path = $r->getFileName();
 
         return <<<EOF
diff --git a/Tests/TestHttpClient.php b/Tests/TestHttpClient.php
@@ -66,7 +66,7 @@ protected function doRequest($request): Response
 
     protected function getScript($request)
     {
-        $r = new \ReflectionClass('Symfony\Component\BrowserKit\Response');
+        $r = new \ReflectionClass(Response::class);
         $path = $r->getFileName();
 
         return <<<EOF

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ public function testGetResponse()`
`88`	`88`	`$client->request('GET', 'http://example.com/');`
`89`	`89`
`90`	`90`	`$this->assertSame('foo', $client->getResponse()->getContent(), '->getCrawler() returns the Response of the last request');`
`91`		`- $this->assertInstanceOf('Symfony\Component\BrowserKit\Response', $client->getResponse(), '->getCrawler() returns the Response of the last request');`
	`91`	`+ $this->assertInstanceOf(Response::class, $client->getResponse(), '->getCrawler() returns the Response of the last request');`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`public function testGetResponseNull()`
`@@ -296,7 +296,7 @@ public function testClickLinkNotFound()`
`296`	`296`	`$client->clickLink('foo');`
`297`	`297`	`$this->fail('->clickLink() throws a \InvalidArgumentException if the link could not be found');`
`298`	`298`	`} catch (\Exception $e) {`
`299`		`- $this->assertInstanceOf('InvalidArgumentException', $e, '->clickLink() throws a \InvalidArgumentException if the link could not be found');`
	`299`	`+ $this->assertInstanceOf(\InvalidArgumentException::class, $e, '->clickLink() throws a \InvalidArgumentException if the link could not be found');`
`300`	`300`	`}`
`301`	`301`	`}`
`302`	`302`
`@@ -355,7 +355,7 @@ public function testSubmitFormNotFound()`
`355`	`355`	`], 'POST');`
`356`	`356`	`$this->fail('->submitForm() throws a \InvalidArgumentException if the form could not be found');`
`357`	`357`	`} catch (\Exception $e) {`
`358`		`- $this->assertInstanceOf('InvalidArgumentException', $e, '->submitForm() throws a \InvalidArgumentException if the form could not be found');`
	`358`	`+ $this->assertInstanceOf(\InvalidArgumentException::class, $e, '->submitForm() throws a \InvalidArgumentException if the form could not be found');`
`359`	`359`	`}`
`360`	`360`	`}`
`361`	`361`
`@@ -406,7 +406,7 @@ public function testFollowRedirect()`
`406`	`406`	`$client->followRedirect();`
`407`	`407`	`$this->fail('->followRedirect() throws a \LogicException if the request was not redirected');`
`408`	`408`	`} catch (\Exception $e) {`
`409`		`- $this->assertInstanceOf('LogicException', $e, '->followRedirect() throws a \LogicException if the request was not redirected');`
	`409`	`+ $this->assertInstanceOf(\LogicException::class, $e, '->followRedirect() throws a \LogicException if the request was not redirected');`
`410`	`410`	`}`
`411`	`411`
`412`	`412`	`$client->setNextResponse(new Response('', 302, ['Location' => 'http://www.example.com/redirected']));`
`@@ -436,7 +436,7 @@ public function testFollowRedirect()`
`436`	`436`	`$client->followRedirect();`
`437`	`437`	`$this->fail('->followRedirect() throws a \LogicException if the request did not respond with 30x HTTP Code');`
`438`	`438`	`} catch (\Exception $e) {`
`439`		`- $this->assertInstanceOf('LogicException', $e, '->followRedirect() throws a \LogicException if the request did not respond with 30x HTTP Code');`
	`439`	`+ $this->assertInstanceOf(\LogicException::class, $e, '->followRedirect() throws a \LogicException if the request did not respond with 30x HTTP Code');`
`440`	`440`	`}`
`441`	`441`	`}`
`442`	`442`
`@@ -466,7 +466,7 @@ public function testFollowRedirectWithMaxRedirects()`
`466`	`466`	`$client->followRedirect();`
`467`	`467`	`$this->fail('->followRedirect() throws a \LogicException if the request was redirected and limit of redirections was reached');`
`468`	`468`	`} catch (\Exception $e) {`
`469`		`- $this->assertInstanceOf('LogicException', $e, '->followRedirect() throws a \LogicException if the request was redirected and limit of redirections was reached');`
	`469`	`+ $this->assertInstanceOf(\LogicException::class, $e, '->followRedirect() throws a \LogicException if the request was redirected and limit of redirections was reached');`
`470`	`470`	`}`
`471`	`471`
`472`	`472`	`$client->setNextResponse(new Response('', 302, ['Location' => 'http://www.example.com/redirected']));`
`@@ -749,7 +749,7 @@ public function testInsulatedRequests()`
`749`	`749`	`$client->request('GET', 'http://www.example.com/foo/foobar');`
`750`	`750`	`$this->fail('->request() throws a \RuntimeException if the script has an error');`
`751`	`751`	`} catch (\Exception $e) {`
`752`		`- $this->assertInstanceOf('RuntimeException', $e, '->request() throws a \RuntimeException if the script has an error');`
	`752`	`+ $this->assertInstanceOf(\RuntimeException::class, $e, '->request() throws a \RuntimeException if the script has an error');`
`753`	`753`	`}`
`754`	`754`	`}`
`755`	`755`
`@@ -837,7 +837,7 @@ public function testInternalRequest()`
`837`	`837`	`'NEW_SERVER_KEY' => 'new-server-key-value',`
`838`	`838`	`]);`
`839`	`839`
`840`		`- $this->assertInstanceOf('Symfony\Component\BrowserKit\Request', $client->getInternalRequest());`
	`840`	`+ $this->assertInstanceOf(\Symfony\Component\BrowserKit\Request::class, $client->getInternalRequest());`
`841`	`841`	`}`
`842`	`842`
`843`	`843`	`public function testInternalRequestNull()`
Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ public function testFromStringWithUrl()`
`104`	`104`
`105`	`105`	`public function testFromStringThrowsAnExceptionIfCookieIsNotValid()`
`106`	`106`	`{`
`107`		`- $this->expectException('InvalidArgumentException');`
	`107`	`+ $this->expectException(\InvalidArgumentException::class);`
`108`	`108`	`Cookie::fromString('foo');`
`109`	`109`	`}`
`110`	`110`
`@@ -117,7 +117,7 @@ public function testFromStringIgnoresInvalidExpiresDate()`
`117`	`117`
`118`	`118`	`public function testFromStringThrowsAnExceptionIfUrlIsNotValid()`
`119`	`119`	`{`
`120`		`- $this->expectException('InvalidArgumentException');`
	`120`	`+ $this->expectException(\InvalidArgumentException::class);`
`121`	`121`	`Cookie::fromString('foo=bar', 'foobar');`
`122`	`122`	`}`
`123`	`123`
`@@ -200,7 +200,7 @@ public function testIsExpired()`
`200`	`200`
`201`	`201`	`public function testConstructException()`
`202`	`202`	`{`
`203`		`- $this->expectException('UnexpectedValueException');`
	`203`	`+ $this->expectException(\UnexpectedValueException::class);`
`204`	`204`	`$this->expectExceptionMessage('The cookie expiration time "string" is not valid.');`
`205`	`205`	`new Cookie('foo', 'bar', 'string');`
`206`	`206`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ protected function doRequest($request): Response`
`43`	`43`
`44`	`44`	`protected function getScript($request)`
`45`	`45`	`{`
`46`		`- $r = new \ReflectionClass('Symfony\Component\BrowserKit\Response');`
	`46`	`+ $r = new \ReflectionClass(Response::class);`
`47`	`47`	`$path = $r->getFileName();`
`48`	`48`
`49`	`49`	`return <<<EOF`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ protected function doRequest($request): Response`
`66`	`66`
`67`	`67`	`protected function getScript($request)`
`68`	`68`	`{`
`69`		`- $r = new \ReflectionClass('Symfony\Component\BrowserKit\Response');`
	`69`	`+ $r = new \ReflectionClass(Response::class);`
`70`	`70`	`$path = $r->getFileName();`
`71`	`71`
`72`	`72`	`return <<<EOF`