BUG#AC-6920:Updated Unit Testcase

Author: rachana

app/code/Magento/QuoteGraphQl/Model/Resolver/MaskedCartId.php

@@ -27,20 +27,22 @@ class MaskedCartId implements ResolverInterface
     /**
      * @var QuoteIdToMaskedQuoteIdInterface
      */
-    private $quoteIdToMaskedQuoteId;
+    private QuoteIdToMaskedQuoteIdInterface $quoteIdToMaskedQuoteId;
 
     /**
      * @var QuoteIdMaskFactory
      */
-    private $quoteIdMaskFactory;
+    private QuoteIdMaskFactory $quoteIdMaskFactory;
 
     /**
      * @var QuoteIdMaskResourceModel
      */
-    private $quoteIdMaskResourceModel;
+    private QuoteIdMaskResourceModel $quoteIdMaskResourceModel;
 
     /**
      * @param QuoteIdToMaskedQuoteIdInterface $quoteIdToMaskedQuoteId
+     * @param QuoteIdMaskFactory $quoteIdMaskFactory
+     * @param QuoteIdMaskResourceModel $quoteIdMaskResourceModel
      */
     public function __construct(
         QuoteIdToMaskedQuoteIdInterface $quoteIdToMaskedQuoteId,
@@ -83,6 +85,7 @@ private function getQuoteMaskId(int $quoteId): string
         }
         return $maskedId;
     }
+
     /**
      * Create masked id for quote if it's not exists
      *

app/code/Magento/QuoteGraphQl/Test/Unit/Model/Resolver/MaskedCartIdTest.php

@@ -16,6 +16,8 @@
 use Magento\Quote\Model\ResourceModel\Quote\QuoteIdMask as QuoteIdMaskResourceModel;
 use Magento\QuoteGraphQl\Model\Resolver\Cart;
 use Magento\QuoteGraphQl\Model\Resolver\MaskedCartId;
+use Magento\Sales\Model\ResourceModel\Order\Collection;
+use Magento\Sales\Model\ResourceModel\Order\CollectionFactory;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
@@ -29,12 +31,12 @@ class MaskedCartIdTest extends TestCase
     /**
      * @var QuoteIdToMaskedQuoteIdInterface|MockObject
      */
-    private QuoteIdToMaskedQuoteIdInterface $quoteIdToMaskedQuoteIdMock;
+    private QuoteIdToMaskedQuoteIdInterface $quoteIdToMaskedQuoteId;
 
     /**
      * @var QuoteIdMaskFactory|MockObject
      */
-    private QuoteIdMaskFactory $quoteIdMaskFactoryMock;
+    private QuoteIdMaskFactory $quoteIdMaskFactory;
 
     /**
      * @var QuoteIdMaskResourceModel|MockObject
@@ -68,16 +70,23 @@ class MaskedCartIdTest extends TestCase
 
     protected function setUp(): void
     {
-        $this->quoteIdToMaskedQuoteIdMock = $this->createMock(QuoteIdToMaskedQuoteIdInterface::class);
-        $this->quoteIdMaskFactoryMock = $this->createMock(QuoteIdMaskFactory::class);
+        $this->quoteIdToMaskedQuoteId = $this->createPartialMock(
+            QuoteIdToMaskedQuoteIdInterface::class,
+            ['execute']
+        );
+        $this->quoteIdMaskFactory = $this->createPartialMock(
+            QuoteIdMaskFactory ::class,
+            ['create']
+        );
+        $this->quoteIdMaskFactory = $this->getMockBuilder(QuoteIdMaskFactory::class)
+            ->disableOriginalConstructor()
+            ->addMethods(['setQuoteId']
+            )
+            ->getMock();
         $this->quoteIdMaskResourceModelMock = $this->createMock(QuoteIdMaskResourceModel::class);
         $this->fieldMock = $this->createMock(Field::class);
         $this->resolveInfoMock = $this->createMock(ResolveInfo::class);
         $this->contextMock = $this->createMock(Context::class);
-        $this->cartMock = $this->getMockBuilder(Cart::class)
-            ->disableOriginalConstructor()
-            ->addMethods(['getId'])
-            ->getMock();
         $this->quoteIdMaskResourceModelMock = $this->getMockBuilder(QuoteIdMaskResourceModel::class)
             ->disableOriginalConstructor()
             ->addMethods(
@@ -87,12 +96,17 @@ protected function setUp(): void
                 ]
             )
             ->getMock();
+        $this->cartMock = $this->getMockBuilder(Cart::class)
+            ->disableOriginalConstructor()
+            ->addMethods(['setQuoteId','getId'])
+            ->getMock();
         $this->maskedCartId = new MaskedCartId(
-            $this->quoteIdToMaskedQuoteIdMock,
-            $this->quoteIdMaskFactoryMock,
+            $this->quoteIdToMaskedQuoteId,
+            $this->quoteIdMaskFactory,
             $this->quoteIdMaskResourceModelMock
         );
     }
+
     public function testResolveWithoutModelInValueParameter(): void
     {
         $this->expectException(LocalizedException::class);
@@ -102,20 +116,19 @@ public function testResolveWithoutModelInValueParameter(): void
 
     public function testResolve(): void
     {
-        $this->valueMock = ['model' => $this->valueMock];
-        $this->cartMock
-            ->expects($this->any())
-            ->method('getId');
-        $this->quoteIdMaskResourceModelMock
-            ->expects($this->any())
-            ->method('getQuoteMaskId')
-            ->willReturn('maskedId');
-        $this->quoteIdMaskResourceModelMock
+        $this->valueMock = ['model' => $this->cartMock];
+        $quoteIdMask = $this->createPartialMock(
+            QuoteIdMaskResourceModel::class,
+            ['setQuoteId']
+        );
+        $this->quoteIdToMaskedQuoteId
             ->expects($this->any())
-            ->method('ensureQuoteMaskExist')
-            ->willReturn('maskedId');
-
+            ->method('execute');
+       // echo get_class($this->quoteIdMaskFactory);die;
+       // $this->quoteIdMaskFactory->setQuoteId('maskId');
+        $this->quoteIdMaskFactory->expects($this->once())
+            ->method('create')
+            ->willReturn($quoteIdMask);
+        $this->maskedCartId->resolve($this->fieldMock, $this->contextMock, $this->resolveInfoMock, $this->valueMock);
     }
 }
-
-