fix CI

thecodingmachine · May 14, 2024 · e7e1756 · e7e1756
1 parent d8a2918
commit e7e1756
Show file tree

Hide file tree

Showing 12 changed files with 59 additions and 27 deletions.
diff --git a/src/EmptyResultIterator.php b/src/EmptyResultIterator.php
@@ -0,0 +1,16 @@
+<?php
+
+namespace TheCodingMachine\TDBM;
+
+use Psr\Log\NullLogger;
+use TheCodingMachine\TDBM\ResultIterator;
+
+class EmptyResultIterator extends ResultIterator
+{
+    protected function __construct()
+    {
+        $this->totalCount = 0;
+        $this->logger = new NullLogger();
+    }
+
+}
diff --git a/src/PageIterator.php b/src/PageIterator.php
@@ -121,7 +121,9 @@ public static function createEmpyIterator(ResultIterator $parentResult): self
     public function getIterator()
     {
         if ($this->innerResultIterator === null) {
-            if ($this->mode === TDBMService::MODE_CURSOR) {
+            if ($this->parentResult instanceof EmptyResultIterator) {
+                $this->innerResultIterator = new EmptyInnerResultIterator();
+            } elseif ($this->mode === TDBMService::MODE_CURSOR) {
                 $this->innerResultIterator = InnerResultIterator::createInnerResultIterator($this->magicSql, $this->parameters, $this->limit, $this->offset, $this->columnDescriptors, $this->objectStorage, $this->className, $this->tdbmService, $this->magicQuery, $this->logger);
             } else {
                 $this->innerResultIterator = InnerResultArray::createInnerResultIterator($this->magicSql, $this->parameters, $this->limit, $this->offset, $this->columnDescriptors, $this->objectStorage, $this->className, $this->tdbmService, $this->magicQuery, $this->logger);

diff --git a/src/QueryFactory/FindObjectsFromRawSqlQueryFactory.php b/src/QueryFactory/FindObjectsFromRawSqlQueryFactory.php
@@ -403,7 +403,7 @@ private function generateGroupedSqlCount(array $parsedSql): array
                 $item['delim'] = ',';
                 $innerColumns[] = $item;
             }
-            $innerColumns[count($innerColumns)-1]['delim'] = false;
+            $innerColumns[count($innerColumns) - 1]['delim'] = false;
             $parsedSql['SELECT'] = $innerColumns;
 
             $parsedSql = [

diff --git a/src/ResultIterator.php b/src/ResultIterator.php
@@ -65,15 +65,15 @@ class ResultIterator implements Result, \ArrayAccess, \JsonSerializable
     private $innerResultIterator;
 
     /** @var int|null */
-    private $totalCount;
+    protected $totalCount;
 
     /** @var int */
     private $mode;
 
     /** @var LoggerInterface */
-    private $logger;
+    protected $logger;
 
-    final private function __construct()
+    protected function __construct()
     {
     }
 
@@ -101,10 +101,7 @@ public static function createResultIterator(QueryFactory $queryFactory, array $p
 
     public static function createEmpyIterator(): self
     {
-        $iterator = new static();
-        $iterator->totalCount = 0;
-        $iterator->logger = new NullLogger();
-        return $iterator;
+        return new EmptyResultIterator();
     }
 
     protected function executeCountQuery(): void

diff --git a/src/Utils/BeanDescriptor.php b/src/Utils/BeanDescriptor.php
@@ -1333,7 +1333,7 @@ private function generateFindByDaoCodeForIndex(Index $index, string $beanNamespa
             $parameter = new ParameterGenerator(ltrim($element->getSafeVariableName(), '$'));
             if (!$first && !($element->isCompulsory() && $index->isUnique())) {
                 $parameterType = '?';
-            //$functionParameter = '?';
+                //$functionParameter = '?';
             } else {
                 $parameterType = '';
                 //$functionParameter = '';
@@ -1765,7 +1765,7 @@ private function generateGetForeignKeys(array $fks): MethodGenerator
      * @param string $indent
      * @return string
      */
-    private function psr2VarExport($var, string $indent=''): string
+    private function psr2VarExport($var, string $indent = ''): string
     {
         if (is_array($var)) {
             $indexed = array_keys($var) === range(0, count($var) - 1);

diff --git a/tests/AbstractTDBMObjectTest.php b/tests/AbstractTDBMObjectTest.php
@@ -42,4 +42,21 @@ public function testEmptyResultIterator()
         $this->assertEquals(0, $d->count());
     }
 
+    public function testEmptyPageIterator()
+    {
+        $a = ResultIterator::createEmpyIterator();
+        $b = $a->take(0, 10);
+        foreach ($b as $empty) {
+            throw new \LogicException("Not supposed to iterate on an empty page iterator.");
+        }
+        $this->assertEquals(0, $b->count());
+        $this->assertEquals([], $b->toArray());
+        $this->assertEquals(0, $b->totalCount());
+        $c = $b->map(function ($foo) {
+        });
+        foreach ($c as $empty) {
+            throw new \LogicException("Not supposed to iterate on an empty iterator.");
+        }
+        $this->assertEquals([], $c->toArray());
+    }
 }
diff --git a/tests/NativeWeakrefObjectStorageTest.php b/tests/NativeWeakrefObjectStorageTest.php
@@ -30,7 +30,7 @@ public function testDanglingPointers(): void
         $objectStorage = new NativeWeakrefObjectStorage();
         $dbRow = $this->createMock(DbRow::class);
 
-        for ($i=0; $i<10001; $i++) {
+        for ($i = 0; $i < 10001; $i++) {
             $objectStorage->set('foo', $i, clone $dbRow);
         }
         $this->assertNull($objectStorage->get('foo', 42));

diff --git a/tests/Performance/ManyToOneBench.php b/tests/Performance/ManyToOneBench.php
@@ -73,18 +73,18 @@ private static function initSchema(Connection $connection): void
             $connection->exec($sqlStmt);
         }
 
-        for ($i = 1; $i<200; $i++) {
+        for ($i = 1; $i < 200; $i++) {
             TDBMAbstractServiceTest::insert($connection, 'countries', [
                 'id' => $i,
                 'label' => 'Country '.$i,
             ]);
         }
 
-        for ($i = 1; $i<1000; $i++) {
+        for ($i = 1; $i < 1000; $i++) {
             TDBMAbstractServiceTest::insert($connection, 'users', [
                 'id' => $i,
                 'name' => 'User '.$i,
-                'country_id' => ($i%199) +1,
+                'country_id' => ($i % 199) + 1,
             ]);
         }
     }

diff --git a/tests/TDBMAbstractServiceTest.php b/tests/TDBMAbstractServiceTest.php
@@ -211,9 +211,9 @@ private static function initSchema(Connection $connection): void
             ->column('manager_id')->references('contact')->null();
 
         $db->table('users')
-            ->addAnnotation('AddTrait', ['name'=>TestUserTrait::class], false)
-            ->addAnnotation('AddTrait', ['name'=>TestOtherUserTrait::class, 'modifiers'=>['\\'.TestOtherUserTrait::class.'::method1 insteadof \\'.TestUserTrait::class, '\\'.TestUserTrait::class.'::method1 as method1renamed']], false)
-            ->addAnnotation('AddTraitOnDao', ['name'=>TestUserDaoTrait::class], false)
+            ->addAnnotation('AddTrait', ['name' => TestUserTrait::class], false)
+            ->addAnnotation('AddTrait', ['name' => TestOtherUserTrait::class, 'modifiers' => ['\\'.TestOtherUserTrait::class.'::method1 insteadof \\'.TestUserTrait::class, '\\'.TestUserTrait::class.'::method1 as method1renamed']], false)
+            ->addAnnotation('AddTraitOnDao', ['name' => TestUserDaoTrait::class], false)
             ->implementsInterface(TestUserInterface::class)
             ->implementsInterfaceOnDao(TestUserDaoInterface::class)
             ->extends('contact')
@@ -721,7 +721,7 @@ private static function initSchema(Connection $connection): void
 
         self::insert($connection, 'tracks', [
             'album_id' => 1,
-            'title' =>'Pigs on the Wing 1',
+            'title' => 'Pigs on the Wing 1',
             // Note: Oracle does not have a TIME column type
             'duration' => $timeType->convertToDatabaseValue(new DateTimeImmutable('1970-01-01 00:01:25'), $connection->getDatabasePlatform()),
         ]);

diff --git a/tests/Utils/Annotation/AnnotationParserTest.php b/tests/Utils/Annotation/AnnotationParserTest.php
@@ -19,7 +19,7 @@ public function testParse(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $column = new Column('foo', Type::getType(Type::STRING), ['comment'=>'@UUID']);
+        $column = new Column('foo', Type::getType(Type::STRING), ['comment' => '@UUID']);
         $table = new Table('bar');
         $annotations = $parser->getColumnAnnotations($column, $table);
 
@@ -40,7 +40,7 @@ public function testParseMultiLine(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $column = new Column('foo', Type::getType(Type::STRING), ['comment'=>"\n@UUID"]);
+        $column = new Column('foo', Type::getType(Type::STRING), ['comment' => "\n@UUID"]);
         $table = new Table('bar');
         $annotations = $parser->getColumnAnnotations($column, $table);
 
@@ -54,7 +54,7 @@ public function testParseMultiAnnotations(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $column = new Column('foo', Type::getType(Type::STRING), ['comment'=>"\n@UUID\n@Autoincrement"]);
+        $column = new Column('foo', Type::getType(Type::STRING), ['comment' => "\n@UUID\n@Autoincrement"]);
         $table = new Table('bar');
         $annotations = $parser->getColumnAnnotations($column, $table);
 
@@ -68,7 +68,7 @@ public function testException(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $table = new Table('bar', [], [], [], 0, ['comment'=>"@UUID\n@UUID"]);
+        $table = new Table('bar', [], [], [], 0, ['comment' => "@UUID\n@UUID"]);
         $annotations = $parser->getTableAnnotations($table);
 
         $this->expectException(TDBMException::class);
@@ -81,7 +81,7 @@ public function testParseParameters(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $table = new Table('bar', [], [], [], 0, ['comment'=>'@UUID("v4")']);
+        $table = new Table('bar', [], [], [], 0, ['comment' => '@UUID("v4")']);
         $annotations = $parser->getTableAnnotations($table);
 
         $annotation = $annotations->findAnnotation(UUID::class);
@@ -94,7 +94,7 @@ public function testParseOldUUID(): void
             'UUID' => UUID::class,
         ]);
         // First generation UUID did not use the Doctrine syntax.
-        $table = new Table('bar', [], [], [], 0, ['comment'=>'@UUID v4']);
+        $table = new Table('bar', [], [], [], 0, ['comment' => '@UUID v4']);
         $annotations = $parser->getTableAnnotations($table);
 
         $annotation = $annotations->findAnnotation(UUID::class);

diff --git a/tests/Utils/DbalUtilsTest.php b/tests/Utils/DbalUtilsTest.php
@@ -17,6 +17,6 @@ public function testGenerateTypes(): void
             'key4' => 1,
         ];
 
-        $this->assertSame(['key2'=>Connection::PARAM_INT_ARRAY, 'key3'=>Connection::PARAM_STR_ARRAY, 'key4'=>ParameterType::INTEGER], DbalUtils::generateTypes($params));
+        $this->assertSame(['key2' => Connection::PARAM_INT_ARRAY, 'key3' => Connection::PARAM_STR_ARRAY, 'key4' => ParameterType::INTEGER], DbalUtils::generateTypes($params));
     }
 }
diff --git a/tests/Utils/DefaultNamingStrategyTest.php b/tests/Utils/DefaultNamingStrategyTest.php
@@ -131,7 +131,7 @@ public function testExceptions(): void
 
     public function testBeanAnnotation(): void
     {
-        $table = new Table('chevaux', [], [], [], 0, ['comment'=>'@Bean(name="Cheval")']);
+        $table = new Table('chevaux', [], [], [], 0, ['comment' => '@Bean(name="Cheval")']);
         $strategy = $this->getDefaultNamingStrategyWithStubTables([$table]);
         $this->assertSame('ChevalDao', $strategy->getDaoClassName('chevaux'));
     }