PHP 8 Support (#39)

* Remove unsed import

* Remove unreachable return

* Add null var card number

* Fix php cs use order

* Fix Tests

* Ignore phpunit test results

* Add php8 support

* Remove unnecessary test

If current month january, always return true

* Revert "Fix php cs use order"

This reverts commit c4a86a2.

# Conflicts:
#	tests/Unit/Cards/BaseCardTests.php

* Fix sort lines

Author: faytekin

.gitignore

@@ -5,3 +5,4 @@ phpunit.xml
 coverage.xml
 test-reports
 composer.lock
+.phpunit.result.cache

composer.json

@@ -4,14 +4,14 @@
     "type": "library",
     "keywords": [ "laravel", "validation", "credit card", "validator" ],
     "require": {
-        "php": "7.*",
+        "php": "7.*|8.*",
         "illuminate/validation": "^5.5|^6.0|^7.0|^8.0",
-        "illuminate/translation": "^5.5|^6.0|^7.0|^8.0"
+        "illuminate/translation": "^5.5|^6.0|^7.0|^8.0",
+        "illuminate/support": "^5.5|^6.0|^7.0|^8.0"
     },
     "require-dev": {
-        "php": ">=7.0.0",
         "friendsofphp/php-cs-fixer": "~2.5",
-        "phpunit/phpunit": "6.2.*",
+        "phpunit/phpunit": "6.2.*|^8.5.8|^9.3.3",
         "orchestra/testbench": "^3.5|^4.0|^5.0"
     },
     "autoload": {
@@ -36,7 +36,6 @@
         }
     ],
     "config": {
-      "platform": {"php": "7.0"},
       "preferred-install": "dist",
       "sort-packages": true
     }

src/CardCvc.php

@@ -3,7 +3,6 @@
 namespace LVR\CreditCard;
 
 use Illuminate\Contracts\Validation\Rule;
-use LVR\CreditCard\Cards\Card;
 
 class CardCvc implements Rule
 {

src/CardExpirationDate.php

@@ -57,8 +57,6 @@ public function passes($attribute, $value)
 
             return false;
         }
-
-        return false;
     }
 
     /**

src/Cards/Card.php

@@ -63,7 +63,7 @@ abstract class Card
     protected $checksum_test;
 
     /**
-     * @var string
+     * @var string|null
      */
     private $card_number;
 

tests/Unit/CardExpirationTest.php

@@ -55,9 +55,9 @@ public function it_checks_expiration_month()
         // Future month
         $this->assertTrue($this->monthValidator(Carbon::now()->addMonth()->month)->passes());
 
-        // Current year, past month
+        // Past year, current month
         $this->assertFalse(
-            $this->monthValidator(Carbon::now()->subMonth()->month, Carbon::now()->year)->passes()
+            $this->monthValidator(Carbon::now()->month, Carbon::now()->subYear()->year)->passes()
         );
 
         // Current year, current month
@@ -157,7 +157,7 @@ protected function yearValidator(string $year)
 
     /**
      * @param string $month
-     *
+     * @param null $year
      * @return mixed
      */
     protected function monthValidator(string $month, $year = null)

tests/Unit/Cards/BaseCardTests.php

@@ -6,6 +6,7 @@
 use Illuminate\Foundation\Http\Middleware\TrimStrings;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Validator;
+use Illuminate\Support\Str;
 use LVR\CreditCard\CardNumber;
 use LVR\CreditCard\Exceptions\CreditCardCharactersException;
 use LVR\CreditCard\Exceptions\CreditCardChecksumException;
@@ -57,7 +58,7 @@ public function it_checks_card_number()
     {
         $this->expectException(CreditCardException::class);
         (new $this->instance)
-            ->setCardNumber(str_random(16))
+            ->setCardNumber(Str::random(16))
             ->isValidCardNumber();
     }
 
@@ -92,7 +93,7 @@ public function it_recognises_invalid_card_numbers()
         $this->withoutMiddleware(ConvertEmptyStringsToNull::class);
 
         collect([
-            str_random(16),
+            Str::random(16),
         ])->each(function ($number) {
             $this->assertTrue(
                 Validator::make(