[3.x] Update PHPUnit to 10 (#130)

* update deprecations

* update phpunit/phpcs

Author: heelc29

Tests/AbstractWebApplicationTest.php

@@ -65,7 +65,7 @@ protected function tearDown(): void
      *
      * @return  \Generator
      */
-    public function getDetectRequestUriData(): \Generator
+    public static function getDetectRequestUriData(): \Generator
     {
         // HTTPS, PHP_SELF, REQUEST_URI, HTTP_HOST, SCRIPT_NAME, QUERY_STRING, (resulting uri)
         yield 'HTTP connection with path in PHP_SELF and query string set in REQUEST_URI' => [
@@ -114,7 +114,7 @@ public function getDetectRequestUriData(): \Generator
      *
      * @return  \Generator
      */
-    public function getRedirectData(): \Generator
+    public static function getRedirectData(): \Generator
     {
         // Note: url, (expected result)
         yield 'with_leading_slash' => ['/foo', 'http://' . self::TEST_HTTP_HOST . '/foo'];
@@ -201,7 +201,7 @@ public function testGetDeprecatedInputReadAccess()
         $object = $this->getMockForAbstractClass(AbstractWebApplication::class);
 
         // Validate default objects unique to the web application are created
-        $this->assertInstanceOf(Input::class, $object->input);
+        $this->assertInstanceOf(Input::class, $object->getInput());
     }
 
     /**
@@ -339,7 +339,7 @@ public function testCompressWithGzipEncoding()
 
         $object = $this->getMockBuilder(AbstractWebApplication::class)
             ->setConstructorArgs([null, null, $mockClient])
-            ->setMethods(['checkHeadersSent'])
+            ->onlyMethods(['checkHeadersSent'])
             ->getMockForAbstractClass();
 
         $object->expects($this->once())
@@ -409,7 +409,7 @@ public function testCompressWithDeflateEncoding()
 
         $object = $this->getMockBuilder(AbstractWebApplication::class)
             ->setConstructorArgs([null, null, $mockClient])
-            ->setMethods(['checkHeadersSent'])
+            ->onlyMethods(['checkHeadersSent'])
             ->getMockForAbstractClass();
 
         $object->expects($this->once())
@@ -473,7 +473,7 @@ public function testCompressWithNoAcceptEncodings()
 
         $object = $this->getMockBuilder(AbstractWebApplication::class)
             ->setConstructorArgs([null, null, $mockClient])
-            ->setMethods(['checkHeadersSent'])
+            ->onlyMethods(['checkHeadersSent'])
             ->getMockForAbstractClass();
 
         // Mock a response.
@@ -531,7 +531,14 @@ public function testCompressWithHeadersSent()
             ['deflate', 'gzip']
         );
 
-        $object = $this->getMockForAbstractClass(AbstractWebApplication::class, [null, null, $mockClient]);
+        $object = $this->getMockBuilder(AbstractWebApplication::class)
+            ->setConstructorArgs([null, null, $mockClient])
+            ->onlyMethods(['checkHeadersSent'])
+            ->getMockForAbstractClass();
+
+        $object->expects($this->once())
+            ->method('checkHeadersSent')
+            ->willReturn(true);
 
         // Mock a response.
         $response = new TextResponse(
@@ -738,7 +745,7 @@ public function testRedirectLegacyBehavior()
         $date                 = new \DateTime('now', new \DateTimeZone('GMT'));
         $object->modifiedDate = $date;
 
-        $object->redirect($url, false);
+        $object->redirect($url, 303);
 
         $this->assertSame(
             self::$headers,
@@ -1161,7 +1168,7 @@ public function testRedirectWithMoved()
         $date                 = new \DateTime('now', new \DateTimeZone('GMT'));
         $object->modifiedDate = $date;
 
-        $object->redirect($url, true);
+        $object->redirect($url, 301);
 
         $this->assertSame(
             self::$headers,

Tests/Stubs/Controller.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  (C) 2018 Open Source Matters, Inc. <https://www.joomla.org>
  * @license    GNU General Public License version 2 or later; see LICENSE

Tests/Stubs/HasArgumentsController.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  (C) 2018 Open Source Matters, Inc. <https://www.joomla.org>
  * @license    GNU General Public License version 2 or later; see LICENSE

composer.json

@@ -52,10 +52,10 @@
         "joomla/session": "^3.0",
         "joomla/test": "^3.0",
         "joomla/uri": "^3.0",
-        "phpunit/phpunit": "^9.5.28",
-        "symfony/phpunit-bridge": "^5.0",
-        "squizlabs/php_codesniffer": "~3.7.1",
-        "rector/rector": "^0.15.21",
+        "phpunit/phpunit": "^10.0",
+        "symfony/phpunit-bridge": "^7.0",
+        "squizlabs/php_codesniffer": "~3.10.2",
+        "rector/rector": "^1.0",
         "phpstan/phpstan": "^1.10.7",
         "phan/phan": "^5.4"
     },

phpunit.xml.dist

@@ -1,9 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/8.5/phpunit.xsd"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
          bootstrap="vendor/autoload.php"
-         cacheResultFile=".phpunit.cache/test-results"
-         verbose="true">
+         cacheDirectory=".phpunit.cache">
     <testsuites>
         <testsuite name="default">
             <directory>Tests</directory>