Fix unit tests

Author: qzminski

.github/workflows/ci.yml

@@ -29,15 +29,15 @@ jobs:
               uses: shivammathur/setup-php@v2
               with:
                   php-version: ${{ matrix.php-version }}
-                  coverage: xdebug
+                  coverage: pcov
 
             - name: Install dependencies
               run: COMPOSER_MEMORY_LIMIT=-1 composer install --dev --no-interaction
 
             - name: Run tests
               run: |
                   mkdir -p build/logs
-                  php vendor/bin/phpunit -c phpunit.xml.dist
+                  php vendor/bin/phpunit -c phpunit.xml.dist --coverage-clover build/logs/clover.xml
 
             - name: Upload coverage
               run: |

rector.php

@@ -0,0 +1,7 @@
+<?php
+
+use Rector\Config\RectorConfig;
+use Rector\PHPUnit\CodeQuality\Rector\Expression\DecorateWillReturnMapWithExpectsMockRector;
+
+return RectorConfig::configure()
+    ->withSkip([DecorateWillReturnMapWithExpectsMockRector::class]);

tests/EventListener/TagManagerListenerTest.php

@@ -70,7 +70,6 @@ public function testOnFieldSaveCallback(): void
 
         $dataContainer = $this->createMock(DataContainer::class);
         $dataContainer
-            ->expects($this->exactly(2))
             ->method('__get')
             ->willReturnMap([
                 ['table', 'tl_table'],
@@ -91,7 +90,6 @@ public function testOnFieldSaveCallbackManagerUnsupported(): void
 
         $dataContainer = $this->createMock(DataContainer::class);
         $dataContainer
-            ->expects($this->exactly(2))
             ->method('__get')
             ->willReturnMap([
                 ['table', 'tl_table'],
@@ -114,7 +112,6 @@ public function testOnOptionsCallback(): void
 
         $dataContainer = $this->createMock(DataContainer::class);
         $dataContainer
-            ->expects($this->exactly(2))
             ->method('__get')
             ->willReturnMap([
                 ['table', 'tl_table'],
@@ -135,7 +132,6 @@ public function testOnOptionsCallbackManagerUnsupported(): void
 
         $dataContainer = $this->createMock(DataContainer::class);
         $dataContainer
-            ->expects($this->exactly(2))
             ->method('__get')
             ->willReturnMap([
                 ['table', 'tl_table'],

tests/Manager/DefaultManagerTest.php

@@ -142,7 +142,6 @@ public function testGetFilterOptionsWithPredefinedTagsSource(): void
 
         $dataContainer = $this->createMock(DataContainer::class);
         $dataContainer
-            ->expects($this->exactly(2))
             ->method('__get')
             ->willReturnMap([
                 ['table', 'tl_table'],
@@ -183,7 +182,6 @@ public function testGetSourceRecordsCountWithPredefinedTagsSource(): void
 
         $dataContainer = $this->createMock(DataContainer::class);
         $dataContainer
-            ->expects($this->exactly(2))
             ->method('__get')
             ->willReturnMap([
                 ['table', 'tl_table'],