fixed CI

Author: Halleck45

.github/workflows/ci.yml

@@ -13,8 +13,7 @@ jobs:
         runs-on: ubuntu-latest
         strategy:
             matrix:
-                php-version: [7.0, 7.1, 7.2, 7.3, 7.4, 8.0, 8.1, 8.2, 8.3, 8.4]
-                #php-version: [5.6, 7.0, 7.1, 7.2, 7.3, 7.4, 8.0, 8.1, 8.2, 8.3, 8.4]
+                php-version: [5.6, 7.0, 7.1, 7.2, 7.3, 7.4, 8.0, 8.1, 8.2, 8.3, 8.4]
                 include:
                     - php-version: nightly
             fail-fast: false
@@ -47,9 +46,6 @@ jobs:
             - name: Run tests (${{ matrix.php-version }})
               run: make test
 
-            - name: Run Docker tests
-              run: docker run --rm test_phpmetrics
-
     build:
         if: github.event_name == 'release' && startsWith(github.ref, 'refs/tags/')
         needs: test

.travis.yml

@@ -4,7 +4,7 @@ include artifacts/Makefile
 
 # Run unit tests
 test:
-	./vendor/bin/phpunit -c phpunit.xml.dist
+	./vendor/bin/phpunit -c phpunit.xml.dist --exclude-group binary
 
 # Compatibility check
 compatibility:

Makefile

@@ -35,7 +35,7 @@ public function testICanEnableIssuerPhp5(): void
         $this->assertStringContainsString('Details', $issuer->log);
         $this->assertStringContainsString('https://github.com/phpmetrics/PhpMetrics/issues/new', $output->output);
         $this->assertStringContainsString('Firstname: Jean-François', $issuer->log);
-        $this->assertStringContainsString('IssuerTest.php (line 26)', $issuer->log);
+        $this->assertStringContainsString('IssuerTest.php (line 29)', $issuer->log);
         $issuer->disable();
     }
 

tests/Component/Issuer/IssuerTest.php

@@ -7,12 +7,12 @@ trait TestCaseCompatible
     {
         public function assertStringContainsString($needle, $haystack, $message = '')
         {
-            $this->assertStringContainsString($needle, $haystack, $message);
+            $this->assertContains($needle, $haystack, $message);
         }
 
         public function assertMatchesRegularExpression($pattern, $string, $message = '')
         {
-            $this->assertMatchesRegularExpression($pattern, $string, $message);
+            $this->assertRegExp($pattern, $string, $message);
         }
     }
 } else {

tests/Polyfill/TestCaseCompatible.php

@@ -1,11 +1,13 @@
 <?php
 namespace Test\Binary;
 
+use PHPUnit\Framework\Attributes\Group;
 use Polyfill\TestCaseCompatible;
 
 /**
  * @group binary
  */
+#[Group('binary')]
 class BinTest extends \PHPUnit\Framework\TestCase
 {
     use TestCaseCompatible;

tests/binary/BinTest.php

@@ -1,11 +1,13 @@
 <?php
 namespace Test\Binary;
 
+use PHPUnit\Framework\Attributes\Group;
 use Polyfill\TestCaseCompatible;
 
 /**
  * @group binary
  */
+#[Group('binary')]
 class PharTest extends \PHPUnit\Framework\TestCase
 {
 

tests/binary/PharTest.php

@@ -1,11 +1,13 @@
 <?php
 namespace Test\Binary;
 
+use PHPUnit\Framework\Attributes\Group;
 use Polyfill\TestCaseCompatible;
 
 /**
  * @group binary
  */
+#[Group('binary')]
 class BinReportTest extends \PHPUnit\Framework\TestCase
 {
     use TestCaseCompatible;

tests/binary/ReportTest.php

@@ -1,34 +1,36 @@
 <?php
 namespace Test\Hal\binary;
 
+use PHPUnit\Framework\Attributes\Group;
 use Polyfill\TestCaseCompatible;
 
 /**
  * @group binary
  */
+#[Group('binary')]
 class StandaloneTest extends \PHPUnit\Framework\TestCase
 {
 
     use TestCaseCompatible;
 
-    private $phar;
+    private $binary;
 
     public function setUp(): void
     {
         parent::setUp();
-        $this->phar = __DIR__ . '/../../releases/phpmetrics-linux-x86_64';
+        $this->binary = __DIR__ . '/../../releases/phpmetrics-linux-x86_64';
     }
 
-    public function testICanRunPhar(): void
+    public function testICanRunStandaloneBinary(): void
     {
-        $command = sprintf('%s --version', $this->phar);
+        $command = sprintf('%s --version', $this->binary);
         $r = shell_exec($command);
         $this->assertStringContainsString('PhpMetrics', $r);
     }
 
     public function testICanProvideOneDirectoryToParse(): void
     {
-        $command = sprintf('%s --exclude="" %s 2>&1', $this->phar, __DIR__ . '/examples/1');
+        $command = sprintf('%s --exclude="" %s 2>&1', $this->binary, __DIR__ . '/examples/1');
         $r = shell_exec($command);
         $this->assertStringContainsString('Object oriented programming', $r);
         $this->assertStringContainsString('LOC', $r);
@@ -39,7 +41,7 @@ public function testICanProvideMultipleDirectoriesToParse(): void
     {
         $command = sprintf(
             '%s --exclude="" %s,%s  2>&1',
-            $this->phar,
+            $this->binary,
             __DIR__ . '/examples/1',
             __DIR__ . '/examples/2'
         );