Test on GH Actions (#260)

* Test on GH Actions * Update composer.json * Update composer.json * Fix tests * Update run-tests.yml * Update run-tests.yml * Update run-tests.yml * Update composer.json
thephpleague · Oct 31, 2022 · 93e55fc · 93e55fc
1 parent dbd8508
commit 93e55fc
Show file tree

Hide file tree

Showing 36 changed files with 89 additions and 37 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -0,0 +1,47 @@
+name: Unit Tests
+
+on:
+  push:
+    branches:
+    - master
+  pull_request:
+    branches:
+    - "*"
+  schedule:
+  - cron: '0 0 * * *'
+
+jobs:
+  php-tests:
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+    env:
+      COMPOSER_NO_INTERACTION: 1
+
+    strategy:
+      matrix:
+        php: [8.1, 8.0, 7.4, 7.3, 7.2]
+        dependency-version: [prefer-lowest, prefer-stable]
+
+    name: P${{ matrix.php }} - ${{ matrix.dependency-version }}
+
+    steps:
+    - name: Checkout code
+      uses: actions/checkout@v2
+
+    - name: Setup PHP
+      uses: shivammathur/setup-php@v2
+      with:
+        php-version: ${{ matrix.php }}
+        coverage: none
+        tools: composer:v2
+
+    - name: Install dependencies
+      run: |
+        composer update --${{ matrix.dependency-version }} --prefer-dist --no-progress
+
+    - name: Execute Unit Tests
+      run: composer test
+
+    - name: Check Code Style
+      run: composer check-style
+      if: ${{ matrix.dependency-version == 'prefer-stable' }}
diff --git a/composer.json b/composer.json
@@ -27,17 +27,22 @@
         "psr-4": { "Omnipay\\PayPal\\" : "src/" }
     },
     "require": {
+        "php": "^7.2|^8.0",
         "omnipay/common": "^3"
     },
     "require-dev": {
-        "omnipay/tests": "^3",
-        "squizlabs/php_codesniffer": "^3",
-        "phpro/grumphp": "^0.14"
+        "omnipay/tests": "^4.1.2",
+        "squizlabs/php_codesniffer": "^3"
     },
     "extra": {
         "branch-alias": {
             "dev-master": "3.0.x-dev"
         }
     },
+    "scripts": {
+        "test": "phpunit",
+        "check-style": "phpcs -p --standard=PSR2 src/",
+        "fix-style": "phpcbf -p --standard=PSR2 src/"
+    },
     "prefer-stable": true
 }
diff --git a/tests/ExpressGatewayTest.php b/tests/ExpressGatewayTest.php
@@ -21,7 +21,7 @@ class ExpressGatewayTest extends GatewayTestCase
      */
     protected $voidOptions;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/ExpressInContextGatewayTest.php b/tests/ExpressInContextGatewayTest.php
@@ -21,7 +21,7 @@ class ExpressInContextGatewayTest extends GatewayTestCase
      */
     protected $voidOptions;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/CaptureRequestTest.php b/tests/Message/CaptureRequestTest.php
@@ -12,7 +12,7 @@ class CaptureRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/ExpressAuthorizeRequestTest.php b/tests/Message/ExpressAuthorizeRequestTest.php
@@ -16,7 +16,7 @@ class ExpressAuthorizeRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/ExpressCompleteAuthorizeRequestTest.php b/tests/Message/ExpressCompleteAuthorizeRequestTest.php
@@ -12,7 +12,7 @@ class ExpressCompleteAuthorizeRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
 

diff --git a/tests/Message/ExpressCompletePurchaseRequestTest.php b/tests/Message/ExpressCompletePurchaseRequestTest.php
@@ -12,7 +12,7 @@ class ExpressCompletePurchaseRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
 

diff --git a/tests/Message/ExpressFetchCheckoutRequestTest.php b/tests/Message/ExpressFetchCheckoutRequestTest.php
@@ -12,7 +12,7 @@ class ExpressFetchCheckoutRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
 

diff --git a/tests/Message/ExpressInContextAuthorizeRequestTest.php b/tests/Message/ExpressInContextAuthorizeRequestTest.php
@@ -15,7 +15,7 @@ class ExpressInContextAuthorizeRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/ExpressTransactionSearchRequestTest.php b/tests/Message/ExpressTransactionSearchRequestTest.php
@@ -12,7 +12,7 @@ class ExpressTransactionSearchRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/ExpressVoidRequestTest.php b/tests/Message/ExpressVoidRequestTest.php
@@ -12,7 +12,7 @@ class ExpressVoidRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/FetchTransactionRequestTest.php b/tests/Message/FetchTransactionRequestTest.php
@@ -12,7 +12,7 @@ class FetchTransactionRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
 

diff --git a/tests/Message/ProAuthorizeRequestTest.php b/tests/Message/ProAuthorizeRequestTest.php
@@ -12,7 +12,7 @@ class ProAuthorizeRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/ProPurchaseRequestTest.php b/tests/Message/ProPurchaseRequestTest.php
@@ -12,7 +12,7 @@ class ProPurchaseRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/RefundRequestTest.php b/tests/Message/RefundRequestTest.php
@@ -12,7 +12,7 @@ class RefundRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
 

diff --git a/tests/Message/RestAuthorizeRequestTest.php b/tests/Message/RestAuthorizeRequestTest.php
@@ -12,7 +12,7 @@ class RestAuthorizeRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/RestCancelSubscriptionRequestTest.php b/tests/Message/RestCancelSubscriptionRequestTest.php
@@ -10,7 +10,7 @@ class RestCancelSubscriptionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestCancelSubscriptionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestCompletePurchaseRequestTest.php b/tests/Message/RestCompletePurchaseRequestTest.php
@@ -14,7 +14,7 @@ class RestCompletePurchaseRequestTest extends TestCase
     private $request;
 
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/RestCompleteSubscriptionRequestTest.php b/tests/Message/RestCompleteSubscriptionRequestTest.php
@@ -10,7 +10,7 @@ class RestCompleteSubscriptionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestCompleteSubscriptionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestCreateCardRequestTest.php b/tests/Message/RestCreateCardRequestTest.php
@@ -13,7 +13,7 @@ class RestCreateCardRequestTest extends TestCase
     /** @var CreditCard */
     protected $card;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/RestCreatePlanRequestTest.php b/tests/Message/RestCreatePlanRequestTest.php
@@ -10,7 +10,7 @@ class RestCreatePlanRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestCreatePlanRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestCreateSubscriptionRequestTest.php b/tests/Message/RestCreateSubscriptionRequestTest.php
@@ -10,7 +10,7 @@ class RestCreateSubscriptionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestCreateSubscriptionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestCreateWebhookRequestTest.php b/tests/Message/RestCreateWebhookRequestTest.php
@@ -11,7 +11,7 @@ final class RestCreateWebhookRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestDeleteCardRequestTest.php b/tests/Message/RestDeleteCardRequestTest.php
@@ -13,7 +13,7 @@ class RestDeleteCardRequestTest extends TestCase
     /** @var CreditCard */
     private $card;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/Message/RestFetchPurchaseRequestTest.php b/tests/Message/RestFetchPurchaseRequestTest.php
@@ -9,7 +9,7 @@ class RestFetchPurchaseRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestFetchPurchaseRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestFetchTransactionRequestTest.php b/tests/Message/RestFetchTransactionRequestTest.php
@@ -9,7 +9,7 @@ class RestFetchTransactionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestFetchTransactionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestListPlanRequestTest.php b/tests/Message/RestListPlanRequestTest.php
@@ -9,7 +9,7 @@ class RestListPlanRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestListPlanRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestListWebhooksRequestTest.php b/tests/Message/RestListWebhooksRequestTest.php
@@ -11,7 +11,7 @@ final class RestListWebhooksRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestReactivateSubscriptionRequestTest.php b/tests/Message/RestReactivateSubscriptionRequestTest.php
@@ -10,7 +10,7 @@ class RestReactivateSubscriptionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestReactivateSubscriptionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestSearchTransactionRequestTest.php b/tests/Message/RestSearchTransactionRequestTest.php
@@ -10,7 +10,7 @@ class RestSearchTransactionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestSearchTransactionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestSuspendSubscriptionRequestTest.php b/tests/Message/RestSuspendSubscriptionRequestTest.php
@@ -10,7 +10,7 @@ class RestSuspendSubscriptionRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestSuspendSubscriptionRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestUpdatePlanRequestTest.php b/tests/Message/RestUpdatePlanRequestTest.php
@@ -10,7 +10,7 @@ class RestUpdatePlanRequestTest extends TestCase
     /** @var \Omnipay\PayPal\Message\RestUpdatePlanRequest */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/Message/RestVerifyWebhookSignatureRequestTest.php b/tests/Message/RestVerifyWebhookSignatureRequestTest.php
@@ -11,7 +11,7 @@ final class RestVerifyWebhookSignatureRequestTest extends TestCase
      */
     private $request;
 
-    public function setUp()
+    public function setUp() : void
     {
         $client = $this->getHttpClient();
         $request = $this->getHttpRequest();

diff --git a/tests/ProGatewayTest.php b/tests/ProGatewayTest.php
@@ -7,7 +7,7 @@
 
 class ProGatewayTest extends GatewayTestCase
 {
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();
 

diff --git a/tests/RestGatewayTest.php b/tests/RestGatewayTest.php
@@ -19,7 +19,7 @@ class RestGatewayTest extends GatewayTestCase
     /** @var array */
     public $subscription_options;
 
-    public function setUp()
+    public function setUp() : void
     {
         parent::setUp();