Enable error php.ini directives in GH workflow (#2113)

cakephp · Oct 9, 2022 · 4664e27 · 4664e27
1 parent f3e4259
commit 4664e27
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -48,6 +48,7 @@ jobs:
       with:
         php-version: ${{ matrix.php-version }}
         coverage: pcov
+        ini-values: zend.assertions=1, error_reporting=-1, display_errors=On
 
     - name: Get composer cache directory
       id: composer-cache
@@ -146,7 +147,7 @@ jobs:
       with:
         php-version: ${{ env.PHP_VERSION }}
         extensions: ${{ env.EXTENSIONS }}
-        ini-values: apc.enable_cli = 1, extension = php_fileinfo.dll
+        ini-values: apc.enable_cli=1, extension=php_fileinfo.dll, zend.assertions=1, error_reporting=-1, display_errors=On
         coverage: pcov
 
     - name: Setup SQLServer

diff --git a/tests/Phinx/Console/Command/CreateTest.php b/tests/Phinx/Console/Command/CreateTest.php
@@ -149,7 +149,7 @@ public function testExecuteWithDuplicateMigrationNames()
 
         $commandTester = new CommandTester($command);
         $commandTester->execute(['command' => $command->getName(), 'name' => 'MyDuplicateMigration']);
-        sleep(1.01); // need at least a second due to file naming scheme
+        time_nanosleep(1, 100000); // need at least a second due to file naming scheme
 
         $this->expectException(InvalidArgumentException::class);
         $this->expectExceptionMessage('The migration class name "MyDuplicateMigration" already exists');
@@ -181,7 +181,7 @@ public function testExecuteWithDuplicateMigrationNamesWithNamespace()
 
         $commandTester = new CommandTester($command);
         $commandTester->execute(['command' => $command->getName(), 'name' => 'MyDuplicateMigration']);
-        sleep(1.01); // need at least a second due to file naming scheme
+        time_nanosleep(1, 100000); // need at least a second due to file naming scheme
 
         $this->expectException(InvalidArgumentException::class);
         $this->expectExceptionMessage('The migration class name "Foo\Bar\MyDuplicateMigration" already exists');