Fix QueryBuilderTest::testBatchInsert() (#283)

Co-authored-by: Sergei Predvoditelev <[email protected]>

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Enh #281: Remove unused code in `Command` class (@vjik)
 - Enh #282: Change property `Schema::$typeMap` to constant `Schema::TYPE_MAP` (@Tigrov)
+- Enh #283: Remove unnecessary check for array type in `Schema::loadTableIndexes()` (@Tigrov)
 
 ## 1.1.0 November 12, 2023
 

src/Schema.php

@@ -251,13 +251,12 @@ protected function loadTableForeignKeys(string $tableName): array
      *
      * @return array Indexes for the given table.
      *
-     * @psalm-return array|IndexConstraint[]
+     * @psalm-return IndexConstraint[]
      */
     protected function loadTableIndexes(string $tableName): array
     {
-        $tableIndexes = $this->loadTableConstraints($tableName, self::INDEXES);
-
-        return is_array($tableIndexes) ? $tableIndexes : [];
+        /** @var IndexConstraint[] */
+        return $this->loadTableConstraints($tableName, self::INDEXES);
     }
 
     /**

tests/QueryBuilderTest.php

@@ -181,9 +181,14 @@ public function testAlterColumn(): void
      * @throws NotSupportedException
      * @throws Throwable
      */
-    public function testBatchInsert(string $table, array $columns, iterable $rows, string $expected): void
-    {
-        parent::testBatchInsert($table, $columns, $rows, $expected);
+    public function testBatchInsert(
+        string $table,
+        array $columns,
+        iterable $rows,
+        string $expected,
+        array $expectedParams = [],
+    ): void {
+        parent::testBatchInsert($table, $columns, $rows, $expected, $expectedParams);
     }
 
     /**