Merge pull request #30 from bigcommerce/support-phpunit-95

PHPMNT-100 Support PHPUnit 9.5 or 10

Author: bc-luke

composer.json

@@ -25,7 +25,7 @@
         "php": "^8.1",
         "bigcommerce/injector": "^4.1",
         "phpspec/prophecy-phpunit": "^2.0",
-        "phpunit/phpunit": " ^10.0"
+        "phpunit/phpunit": "^9.5 || ^10.0"
     },
     "require-dev": {
         "phpstan/phpstan": "^1.10"

tests/AutoMockingTestTest.php

@@ -15,7 +15,8 @@ public function testTestShouldNotBeRiskyWhenItHasProphecyExpectations(): void
 
         $riskyTest->runBare();
 
-        $this->assertSame(1, $riskyTest->numberOfAssertionsPerformed());
+        $this->assertFalse($riskyTest->isRisky());
+
     }
 
     public function testTestShouldBeRiskyWhenItHasNoAssertions(): void
@@ -24,6 +25,6 @@ public function testTestShouldBeRiskyWhenItHasNoAssertions(): void
 
         $riskyTest->runBare();
 
-        $this->assertSame(0, $riskyTest->numberOfAssertionsPerformed());
+        $this->assertTrue($riskyTest->isRisky());
     }
 }

tests/Dummy/DummyTest.php

@@ -3,7 +3,9 @@
 
 namespace Tests\Dummy;
 
+use BadMethodCallException;
 use Bigcommerce\MockInjector\AutoMockingTest;
+use Exception;
 use Prophecy\Prophecy\ObjectProphecy;
 
 /**
@@ -33,4 +35,17 @@ public function testWithProphecyExpectations() : void
     public function testWithoutAssertions() : void
     {
     }
+
+    public function isRisky(): bool
+    {
+        if (is_callable([$this, 'numberOfAssertionsPerformed'])) {
+            return $this->numberOfAssertionsPerformed() === 0;
+        }
+
+        if (is_callable([$this, 'getNumAssertions'])) {
+            return $this->getNumAssertions() === 0;
+        }
+
+        throw new BadMethodCallException("Could not check the number of assertions performed");
+    }
 }