Merge pull request #560 from flightphp/php83-fixes

phpunit fixes for deprecated notices. Also Collection reset removal

Author: n0nag0n

flight/database/PdoWrapper.php

@@ -49,7 +49,8 @@ public function runQuery(string $sql, array $params = []): PDOStatement
     public function fetchField(string $sql, array $params = [])
     {
         $result = $this->fetchRow($sql, $params);
-        return reset($result);
+        $data = $result->getData();
+        return reset($data);
     }
 
     /**

flight/util/Collection.php

@@ -20,18 +20,6 @@
  */
 class Collection implements ArrayAccess, Iterator, Countable, JsonSerializable
 {
-    /**
-     * This is to allow for reset() to work properly.
-     *
-     * WARNING! This MUST be the first variable in this class!!!
-     *
-     * PHPStan is ignoring this because we don't need actually to read the property
-     *
-     * @var mixed
-     * @phpstan-ignore-next-line
-     */
-    private $first_property = null;
-
     /**
      * Collection data.
      *
@@ -47,7 +35,6 @@ class Collection implements ArrayAccess, Iterator, Countable, JsonSerializable
     public function __construct(array $data = [])
     {
         $this->data = $data;
-        $this->handleReset();
     }
 
     /**
@@ -68,7 +55,6 @@ public function __get(string $key)
     public function __set(string $key, $value): void
     {
         $this->data[$key] = $value;
-        $this->handleReset();
     }
 
     /**
@@ -85,7 +71,6 @@ public function __isset(string $key): bool
     public function __unset(string $key): void
     {
         unset($this->data[$key]);
-        $this->handleReset();
     }
 
     /**
@@ -115,7 +100,6 @@ public function offsetSet($offset, $value): void
         } else {
             $this->data[$offset] = $value;
         }
-        $this->handleReset();
     }
 
     /**
@@ -136,17 +120,6 @@ public function offsetExists($offset): bool
     public function offsetUnset($offset): void
     {
         unset($this->data[$offset]);
-        $this->handleReset();
-    }
-
-    /**
-     * This is to allow for reset() of a Collection to work properly.
-     *
-     * @return void
-     */
-    public function handleReset()
-    {
-        $this->first_property = reset($this->data);
     }
 
     /**
@@ -234,7 +207,6 @@ public function getData(): array
     public function setData(array $data): void
     {
         $this->data = $data;
-        $this->handleReset();
     }
 
     #[\ReturnTypeWillChange]

phpunit.xml

@@ -23,6 +23,7 @@
     </testsuites>
     <logging />
     <php>
+        <ini name="error_reporting" value="-1"/>
         <env name="PHPUNIT_TEST" value="true" force="true" />
     </php>
 </phpunit>

tests/CollectionTest.php

@@ -99,27 +99,4 @@ public function testClear()
         $this->collection->clear();
         $this->assertEquals(0, $this->collection->count());
     }
-
-    public function testResetByProperty()
-    {
-        $this->collection->a = 11;
-        $this->collection->b = 22;
-        $result = reset($this->collection);
-        $this->assertEquals(11, $result);
-    }
-
-    public function testResetBySetData()
-    {
-        $this->collection->setData(['a' => 11, 'b' => 22]);
-        $result = reset($this->collection);
-        $this->assertEquals(11, $result);
-    }
-
-    public function testResetByArraySet()
-    {
-        $this->collection['a'] = 11;
-        $this->collection['b'] = 22;
-        $result = reset($this->collection);
-        $this->assertEquals(11, $result);
-    }
 }

tests/DispatcherTest.php

@@ -305,7 +305,7 @@ public function testExecuteStringClassDefaultContainerArraySyntax(): void
     public function testExecuteStringClassDefaultContainerButForgotInjectingEngine(): void
     {
         $this->expectException(TypeError::class);
-        $this->expectExceptionMessageMatches('#Argument 1 passed to tests\\\\classes\\\\ContainerDefault::__construct\(\) must be an instance of flight\\\\Engine, null given#');
+        $this->expectExceptionMessageMatches('#tests\\\\classes\\\\ContainerDefault::__construct\(\).+flight\\\\Engine, null given#');
         $result = $this->dispatcher->execute([ ContainerDefault::class, 'testTheContainer' ]);
     }
 }

tests/EngineTest.php

@@ -4,6 +4,7 @@
 
 namespace tests;
 
+use ErrorException;
 use Exception;
 use flight\database\PdoWrapper;
 use flight\Engine;
@@ -750,8 +751,8 @@ public function testContainerDicePdoWrapperTestBadParams() {
         $engine->route('/container', Container::class.'->testThePdoWrapper');
         $engine->request()->url = '/container';
 
-        $this->expectException(PDOException::class);
-        $this->expectExceptionMessage("invalid data source name");
+        $this->expectException(ErrorException::class);
+        $this->expectExceptionMessageMatches("/Passing null to parameter/");
 
         $engine->start();
     }