Dropped PHP<7.2 support. Fixed phpunit tests.

Author: ekyna

Diff for: .github/workflows/build.yml

@@ -15,11 +15,8 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php: [7.1, 7.2, 7.3, 7.4, 8.0]
+                php: [7.2, 7.3, 7.4, 8.0]
                 symfony: [^3.4, ^4.4, ^5.2]
-                exclude:
-                    - php: 7.1
-                      symfony: ^5.2
 
         env:
             APP_ENV: test

Diff for: .gitignore

@@ -1,2 +1,3 @@
 /vendor
-/composer.lock
+/composer.lock
+/.phpunit.result.cache

Diff for: composer.json

@@ -1,11 +1,11 @@
 {
-    "name" : "ekyna/payum-monetico",
-    "description" : "Payum Monetico (Credit Mutuel/CIC) gateway",
-    "type" : "component",
-    "authors" : [
+    "name": "ekyna/payum-monetico",
+    "description": "Payum Monetico (Credit Mutuel/CIC) gateway",
+    "type": "component",
+    "authors": [
         {
-            "name" : "Etienne Dauvergne",
-            "homepage" : "http://ekyna.com"
+            "name": "Etienne Dauvergne",
+            "homepage": "https://ekyna.com"
         }
     ],
     "keywords": [
@@ -15,28 +15,23 @@
         "credit mutuel",
         "cic"
     ],
-    "homepage" : "https://github.com/ekyna/PayumMonetico",
-    "license" : "MIT",
-    "require" : {
-        "php" : "^7.1|^8.0",
+    "homepage": "https://github.com/ekyna/PayumMonetico",
+    "license": "MIT",
+    "require": {
+        "php": "^7.2|^8.0",
         "ext-json": "*",
         "payum/core": "^1.5",
-        "psr/log": "~1.0|~2.0",
-        "symfony/options-resolver": "~3.0|~4.0|~5.0",
-        "sokil/php-isocodes": "~2.0|~3.0"
+        "psr/log": "^1.0|^2.0",
+        "symfony/options-resolver": "^3.4|^4.4|^5.2",
+        "sokil/php-isocodes": "^2.0|^3.0"
     },
     "require-dev": {
         "php-http/guzzle6-adapter": "^1.1|^2.0",
-        "phpunit/phpunit": "^7|^8|^9"
+        "phpunit/phpunit": "^7.0|^8.0|^9.0"
     },
-    "autoload" : {
-        "psr-4" : {
-            "Ekyna\\Component\\Payum\\Monetico\\" : "src/"
-        }
-    },
-    "extra": {
-        "branch-alias": {
-            "dev-master": "1.5.x-dev"
+    "autoload": {
+        "psr-4": {
+            "Ekyna\\Component\\Payum\\Monetico\\": "src/"
         }
     }
 }

Diff for: phpunit.xml

@@ -1,28 +1,19 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit backupGlobals="true"
-         backupStaticAttributes="false"
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
          colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnFailure="false"
-         syntaxCheck="false"
          bootstrap="tests/bootstrap.php">
 
+    <coverage processUncoveredFiles="true">
+        <include>
+            <directory suffix=".php">./src</directory>
+        </include>
+    </coverage>
+
     <testsuites>
         <testsuite name="Ekyna Payum Monetico Component Test Suite">
             <directory>./tests</directory>
         </testsuite>
     </testsuites>
 
-    <filter>
-        <whitelist>
-            <directory>./src</directory>
-            <exclude>
-                <directory>./tests</directory>
-                <directory>./vendor</directory>
-            </exclude>
-        </whitelist>
-    </filter>
 </phpunit>

Diff for: tests/Action/AbstractActionTest.php

@@ -15,16 +15,6 @@
  */
 abstract class AbstractActionTest extends GenericActionTest
 {
-    /**
-     * @test
-     */
-    public function couldBeConstructedWithoutAnyArguments()
-    {
-        parent::couldBeConstructedWithoutAnyArguments();
-
-        $this->assertTrue(true);
-    }
-
     /**
      * @return MockObject&GatewayInterface
      */
@@ -48,4 +38,4 @@ protected function createApiMock()
     {
         return $this->getMockBuilder(Api::class)->getMock();
     }
-}
+}

Diff for: tests/Action/Api/PaymentFormActionTest.php

@@ -23,17 +23,6 @@ protected function setUp(): void
         $this->action = new $this->actionClass('template');
     }
 
-    /**
-     * @test
-     */
-    public function couldBeConstructedWithoutAnyArguments()
-    {
-        // TODO Remove this test...
-        //$this->markTestSkipped();
-        $action = new $this->actionClass('template');
-        $this->assertInstanceOf(PaymentFormAction::class, $action);
-    }
-
     /**
      * @test
      */