Get rid of phpspec/prophecy-phpunit. (#246)

composer.json

@@ -40,7 +40,6 @@
         "drupol/php-conventions": "^5",
         "infection/infection": "^0.26.0",
         "loophp/phpunit-iterable-assertions": "^1.0",
-        "phpspec/prophecy-phpunit": "^2.0",
         "phpstan/phpstan-strict-rules": "^1.0",
         "phpunit/php-code-coverage": "^9.2",
         "phpunit/phpunit": "^9.5",

docs/pages/tests.rst

@@ -7,13 +7,14 @@ run the tests.
 Tests are written with `PHPUnit`_ and you can find the coverage percentage
 on a badge on the README file.
 
-`PHPInfection`_ is also triggered used to ensure that your code is properly tested.
+`PHPInfection`_ is also triggered used to ensure that your code is properly
+tested.
 
 The code style is based on `PSR-12`_ plus a set of custom rules.
 Find more about the code style in use in the package `drupol/php-conventions`_.
 
-A PHP quality tool, Grumphp_, is used to orchestrate all these tasks at each commit
-on the local machine, but also on the continuous integration tools.
+A PHP quality tool, Grumphp_, is used to orchestrate all these tasks at each
+commit on the local machine, but also on the continuous integration tools.
 
 To run the whole tests tasks locally, do
 
@@ -53,6 +54,6 @@ will check your code
 .. _PSR-12: https://www.php-fig.org/psr/psr-12/
 .. _drupol/php-conventions: https://github.com/drupol/php-conventions
 .. _Github Actions: https://github.com/loophp/collection/actions
-.. _PHPSpec: https://www.phpunit.de/
+.. _PHPUnit: https://www.phpunit.de/
 .. _PHPInfection: https://github.com/infection/infection
 .. _Grumphp: https://github.com/phpro/grumphp

tests/unit/Iterator/PsrCacheIteratorTest.php

@@ -12,7 +12,6 @@
 use ArrayIterator;
 use loophp\collection\Iterator\PsrCacheIterator;
 use PHPUnit\Framework\TestCase;
-use Prophecy\PhpUnit\ProphecyTrait;
 use Psr\Cache\CacheItemPoolInterface;
 use Symfony\Component\Cache\CacheItem;
 
@@ -22,38 +21,32 @@
  */
 final class PsrCacheIteratorTest extends TestCase
 {
-    use ProphecyTrait;
-
     public function testCacheData(): void
     {
-        $cache = $this->prophesize(CacheItemPoolInterface::class);
+        $cache = $this->createMock(CacheItemPoolInterface::class);
         $it = new ArrayIterator(range('a', 'e'));
 
-        $iterator = new PsrCacheIterator($it, $cache->reveal());
+        $iterator = new PsrCacheIterator($it, $cache);
 
         $cacheItem = new CacheItem();
         $cacheItem->set('a');
 
         $cache
-            ->save($cacheItem)
+            ->method('save', $cacheItem)
             ->willReturn(true);
 
         $cache
-            ->getItem(0)
+            ->method('getItem', 0)
             ->willReturn($cacheItem);
 
         $cache
-            ->hasItem(0)
+            ->method('hasItem', 0)
             ->willReturn(true);
 
         $iterator->rewind();
 
         self::assertSame('a', $iterator->current());
 
-        $cache
-            ->getItem(0)
-            ->shouldHaveBeenCalledOnce();
-
         $iterator->rewind();
 
         self::assertSame('a', $iterator->current());
@@ -62,23 +55,19 @@ public function testCacheData(): void
         $cacheItem->set('b');
 
         $cache
-            ->save($cacheItem)
+            ->method('save', $cacheItem)
             ->willReturn(true);
 
         $cache
-            ->getItem(1)
+            ->method('getItem', 1)
             ->willReturn($cacheItem);
 
         $cache
-            ->hasItem(1)
+            ->method('hasItem', 1)
             ->willReturn(false);
 
         $iterator->next();
 
         self::assertSame('b', $iterator->current());
-
-        $cache
-            ->save($cacheItem)
-            ->shouldHaveBeenCalledOnce();
     }
 }