fix: address Copilot review comments on PR #286

- Use require_once instead of include_once in loader.php for fail-fast
- Fix array indentation in SessionTest.php
- Fix docblock formatting in SnowflakeDatabaseReflectionTest and SnowflakeTableQueryBuilderTest

Author: zajca

tests/Functional/Bigquery/Connection/SessionTest.php

@@ -29,15 +29,15 @@ public function testCreateSession(): void
             $this->bqClient->query(
                 'SELECT 1',
                 [
-                'configuration' => [
-                'query' => [
-                    'connectionProperties' => [
-                        'key' => 'session_id',
-                        'value' => $session->getSessionId(),
+                    'configuration' => [
+                        'query' => [
+                            'connectionProperties' => [
+                                'key' => 'session_id',
+                                'value' => $session->getSessionId(),
+                            ],
+                        ],
                     ],
                 ],
-                ],
-                ],
             ),
         );
 

tests/Functional/Snowflake/Database/SnowflakeDatabaseReflectionTest.php

@@ -17,9 +17,7 @@ class SnowflakeDatabaseReflectionTest extends SnowflakeBaseCase
     protected function setUp(): void
     {
         $this->markTestSkipped('Test won\'t work as testing user has not such high privileges.');
-        /**
- * @phpstan-ignore-next-line
-*/
+        /** @phpstan-ignore-next-line */
         parent::setUp();
         $this->setUpUser(self::USERNAME);
         $this->setUpRole(self::ROLE);

tests/Unit/Table/Snowflake/SnowflakeTableQueryBuilderTest.php

@@ -143,49 +143,37 @@ public static function provideGetColumnDefinitionUpdate(): Generator
         yield 'drop default' => [
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => true, 'default' => '10']),
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => true, 'default' => null]),
-            /**
- * @lang Snowflake
-*/
+            /** @lang Snowflake */
             'ALTER TABLE "testDb"."testTable" MODIFY COLUMN "testColumn" DROP DEFAULT',
         ];
         yield 'add nullable' => [
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => false, 'default' => '']),
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => true, 'default' => '']),
-            /**
- * @lang Snowflake
-*/
+            /** @lang Snowflake */
             'ALTER TABLE "testDb"."testTable" MODIFY COLUMN "testColumn" DROP NOT NULL',
         ];
         yield 'drop nullable' => [
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => true, 'default' => '']),
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => false, 'default' => '']),
-            /**
- * @lang Snowflake
-*/
+            /** @lang Snowflake */
             'ALTER TABLE "testDb"."testTable" MODIFY COLUMN "testColumn" SET NOT NULL',
         ];
         yield 'increase length of text column' => [
             new Snowflake('VARCHAR', ['length' => '12', 'nullable' => true, 'default' => '']),
             new Snowflake('VARCHAR', ['length' => '38', 'nullable' => true, 'default' => '']),
-            /**
- * @lang Snowflake
-*/
+            /** @lang Snowflake */
             'ALTER TABLE "testDb"."testTable" MODIFY COLUMN "testColumn" SET DATA TYPE VARCHAR(38)',
         ];
         yield 'increase precision of numeric column' => [
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => true, 'default' => '']),
             new Snowflake('NUMERIC', ['length' => '14,8', 'nullable' => true, 'default' => '']),
-            /**
- * @lang Snowflake
-*/
+            /** @lang Snowflake */
             'ALTER TABLE "testDb"."testTable" MODIFY COLUMN "testColumn" SET DATA TYPE NUMERIC(14, 8)',
         ];
         yield 'full set of changes (increase precision, drop nullable, drop default)' => [
             new Snowflake('NUMERIC', ['length' => '12,8', 'nullable' => true, 'default' => 'grunbread']),
             new Snowflake('NUMERIC', ['length' => '14,8', 'nullable' => false, 'default' => '']),
-            /**
- * @lang Snowflake
-*/
+            /** @lang Snowflake */
             'ALTER TABLE "testDb"."testTable" MODIFY COLUMN "testColumn" DROP DEFAULT, '
             . 'COLUMN "testColumn" SET NOT NULL, COLUMN "testColumn" SET DATA TYPE NUMERIC(14, 8)',
         ];