Added missing dependency to unit test to get the tests to pass

Author: digitalrisedorset

lib/internal/Magento/Framework/App/Test/Unit/PageCache/IdentifierTest.php

@@ -12,6 +12,7 @@
 use Magento\Framework\App\Request\Http as HttpRequest;
 use Magento\Framework\Serialize\Serializer\Json;
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
+use Magento\PageCache\Model\App\Request\Http\IdentifierStoreReader;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
@@ -46,6 +47,10 @@ class IdentifierTest extends TestCase
      * @var Json|MockObject
      */
     private $serializerMock;
+    /**
+     * @var IdentifierStoreReader|MockObject
+     */
+    private $identifierStoreReader;
 
     /**
      * @inheritdoc
@@ -73,12 +78,19 @@ function ($value) {
                 }
             );
 
+        $this->identifierStoreReader = $this->getMockBuilder(IdentifierStoreReader::class)
+            ->onlyMethods(['getPageTagsWithStoreCacheTags'])
+            ->disableOriginalConstructor()
+            ->getMock();
+
+
         $this->model = $this->objectManager->getObject(
             Identifier::class,
             [
                 'request'    => $this->requestMock,
                 'context'    => $this->contextMock,
-                'serializer' => $this->serializerMock
+                'serializer' => $this->serializerMock,
+                'identifierStoreReader' => $this->identifierStoreReader
             ]
         );
         parent::setUp();
@@ -96,6 +108,12 @@ public function testSecureDifferentiator(): void
             ->willReturn('http://example.com/path/');
         $this->contextMock->method('getVaryString')->willReturn(self::VARY);
 
+        $this->identifierStoreReader->method('getPageTagsWithStoreCacheTags')->willReturnCallback(
+            function ($value) {
+                return $value;
+            }
+        );
+
         $valueWithSecureRequest = $this->model->getValue();
         $valueWithInsecureRequest = $this->model->getValue();
         $this->assertNotEquals($valueWithSecureRequest, $valueWithInsecureRequest);
@@ -112,6 +130,12 @@ public function testDomainDifferentiator(): void
             ->willReturnOnConsecutiveCalls('http://example.com/path/', 'http://example.net/path/');
         $this->contextMock->method('getVaryString')->willReturn(self::VARY);
 
+        $this->identifierStoreReader->method('getPageTagsWithStoreCacheTags')->willReturnCallback(
+            function ($value) {
+                return $value;
+            }
+        );
+
         $valueDomain1 = $this->model->getValue();
         $valueDomain2 = $this->model->getValue();
         $this->assertNotEquals($valueDomain1, $valueDomain2);
@@ -128,6 +152,12 @@ public function testPathDifferentiator(): void
             ->willReturnOnConsecutiveCalls('http://example.com/path/', 'http://example.com/path1/');
         $this->contextMock->method('getVaryString')->willReturn(self::VARY);
 
+        $this->identifierStoreReader->method('getPageTagsWithStoreCacheTags')->willReturnCallback(
+            function ($value) {
+                return $value;
+            }
+        );
+
         $valuePath1 = $this->model->getValue();
         $valuePath2 = $this->model->getValue();
         $this->assertNotEquals($valuePath1, $valuePath2);
@@ -173,6 +203,12 @@ public function testGetValue(): void
             ->method('getVaryString')
             ->willReturn(self::VARY);
 
+        $this->identifierStoreReader->method('getPageTagsWithStoreCacheTags')->willReturnCallback(
+            function ($value) {
+                return $value;
+            }
+        );
+
         $this->assertEquals(
             sha1(
                 json_encode(