Fix the build (#90)

Fix the code quality and fix the PHPUnit issue with PHPUnit 10.5.

.github/workflows/testing-and-cs.yaml

@@ -36,7 +36,7 @@ jobs:
           tools: composer:${{ matrix.composer }}
       - run: composer require "symfony/finder:${{ matrix.symfony }}" "symfony/filesystem:${{ matrix.symfony }}"
       - run: composer install
-      - run: vendor/bin/phpunit tests
+      - run: vendor/bin/phpunit
   PHPUnit-Symfony56:
     name: PHPUnit testing
     runs-on: ubuntu-latest
@@ -56,7 +56,7 @@ jobs:
           tools: composer:${{ matrix.composer }}
       - run: composer require "symfony/finder:${{ matrix.symfony }}" "symfony/filesystem:${{ matrix.symfony }}"
       - run: composer install
-      - run: vendor/bin/phpunit tests
+      - run: vendor/bin/phpunit
 
   BuildDone:
     name: PHP full build

phpunit.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
+         bootstrap="vendor/autoload.php"
+         failOnRisky="true"
+         failOnWarning="true">
+    <testsuites>
+        <testsuite name="default">
+            <directory>tests</directory>
+            <exclude>tests/ZipArchiverTest/TypicalArchive</exclude>
+            <exclude>tests/ZipArchiverTest/ComposerJsonArchive</exclude>
+        </testsuite>
+    </testsuites>
+
+    <source restrictDeprecations="true" restrictNotices="true" restrictWarnings="true">
+        <include>
+            <directory>src</directory>
+        </include>
+    </source>
+</phpunit>

src/Configuration.php

@@ -303,13 +303,13 @@ private function parseNexusExtra(InputInterface $input, Composer $composer)
             }
         } else {
             // configurations in composer.json support upload to multi repository
-            foreach ($extras[$extrasConfigurationKey] as $key=> $nexusPushConfigItem) {
+            foreach ($extras[$extrasConfigurationKey] as $key => $nexusPushConfigItem) {
                 if (empty($nexusPushConfigItem[self::PUSH_CFG_NAME])) {
                     $fmt = 'The push configuration array in composer.json with index {%s} need provide value for key "%s"';
                     $exceptionMsg = sprintf($fmt, $key, self::PUSH_CFG_NAME);
                     throw new InvalidConfigException($exceptionMsg);
                 }
-                if ($nexusPushConfigItem[self::PUSH_CFG_NAME] ==$repository) {
+                if ($nexusPushConfigItem[self::PUSH_CFG_NAME] == $repository) {
                     return $nexusPushConfigItem;
                 }
             }

src/ZipArchiver.php

@@ -128,7 +128,7 @@ private static function updateVersion($zipFile, $subDirectory, $version, $io)
 
         $archive->deleteName($filePath);
 
-        $archive->addFromString($filePath, json_encode($content, JSON_PRETTY_PRINT|JSON_UNESCAPED_SLASHES));
+        $archive->addFromString($filePath, json_encode($content, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES));
 
         $archive->close();
     }

tests/ZipArchiverTest.php

@@ -39,7 +39,7 @@ public function testArchiveDirectory(string $directory, array $expectedResult, s
         $this->assertArchiveContainsFiles($this->generationPath, $expectedResult);
     }
 
-    public function zipArchiverProvider() {
+    public static function zipArchiverProvider() {
         return [
             [
                 __DIR__ . '/ZipArchiverTest/TypicalArchive',
@@ -103,7 +103,7 @@ public function testComposerArchiveDirectory(string $directory, array $expectedR
         $this->assertComposerJsonVersion($this->generationPath, $subdirectory, $version);
     }
 
-    public function composerArchiverProvider() {
+    public static function composerArchiverProvider() {
         return [
             [
                 __DIR__ . '/ZipArchiverTest/ComposerJsonArchive',