#39169: Special Price To Date is wrongly validated on applySpecialPrice

- fix unit tests

Author: KrasnoshchokBohdan

app/code/Magento/Bundle/Model/Product/Price.php

@@ -52,12 +52,6 @@ class Price extends \Magento\Catalog\Model\Product\Type\Price
      */
     private $serializer;
 
-
-    /**
-     * @var SpecialPriceService
-     */
-    protected SpecialPriceService $specialPriceService;
-
     /**
      * Constructor
      *
@@ -94,8 +88,6 @@ public function __construct(
         $this->_catalogData = $catalogData;
         $this->serializer = $serializer ?: ObjectManager::getInstance()
             ->get(\Magento\Framework\Serialize\Serializer\Json::class);
-        $this->specialPriceService = $specialPriceService ?: ObjectManager::getInstance()
-            ->get(SpecialPriceService::class);
         parent::__construct(
             $ruleFactory,
             $storeManager,
@@ -106,7 +98,8 @@ public function __construct(
             $groupManagement,
             $tierPriceFactory,
             $config,
-            $tierPriceExtensionFactory
+            $tierPriceExtensionFactory,
+            $specialPriceService
         );
     }
 

app/code/Magento/Bundle/Test/Unit/Model/Product/PriceTest.php

@@ -1,7 +1,7 @@
 <?php
 /**
- * Copyright © Magento, Inc. All rights reserved.
- * See COPYING.txt for license details.
+ * Copyright 2015 Adobe
+ * All Rights Reserved.
  */
 declare(strict_types=1);
 
@@ -14,6 +14,7 @@
 use Magento\Catalog\Api\Data\ProductTierPriceInterfaceFactory;
 use Magento\Catalog\Helper\Data;
 use Magento\Catalog\Model\Product;
+use Magento\Catalog\Service\SpecialPriceService;
 use Magento\CatalogRule\Model\ResourceModel\RuleFactory;
 use Magento\Customer\Api\GroupManagementInterface;
 use Magento\Customer\Model\Session;
@@ -135,6 +136,17 @@ function ($value) {
             ->onlyMethods(['create'])
             ->disableOriginalConstructor()
             ->getMock();
+
+        $specialPriceService = $this->getMockBuilder(SpecialPriceService::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $specialPriceService->expects($this->any())
+            ->method('execute')
+            ->willReturnCallback(function($value) {
+                return $value;
+            });
+
         $objectManagerHelper = new ObjectManagerHelper($this);
         $this->model = $objectManagerHelper->getObject(
             Price::class,
@@ -150,7 +162,8 @@ function ($value) {
                 'config' => $scopeConfig,
                 'catalogData' => $this->catalogHelperMock,
                 'serializer' => $this->serializer,
-                'tierPriceExtensionFactory' => $tierPriceExtensionFactoryMock
+                'tierPriceExtensionFactory' => $tierPriceExtensionFactoryMock,
+                'specialPriceService' => $specialPriceService
             ]
         );
     }

app/code/Magento/Bundle/Test/Unit/Pricing/Price/SpecialPriceTest.php

@@ -11,6 +11,7 @@
 use Magento\Bundle\Pricing\Price\SpecialPrice;
 use Magento\Catalog\Model\Product;
 use Magento\Catalog\Pricing\Price\RegularPrice;
+use Magento\Catalog\Service\SpecialPriceService;
 use Magento\Framework\Pricing\Price\PriceInterface;
 use Magento\Framework\Pricing\PriceCurrencyInterface;
 use Magento\Framework\Pricing\PriceInfo\Base;
@@ -48,6 +49,11 @@ class SpecialPriceTest extends TestCase
      */
     protected $priceCurrencyMock;
 
+    /**
+     * @var SpecialPriceService|MockObject
+     */
+    private $specialPriceService;
+
     protected function setUp(): void
     {
         $this->saleable = $this->getMockBuilder(Product::class)
@@ -63,13 +69,18 @@ protected function setUp(): void
 
         $this->priceCurrencyMock = $this->getMockForAbstractClass(PriceCurrencyInterface::class);
 
+        $this->specialPriceService = $this->getMockBuilder(SpecialPriceService::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
         $objectHelper = new ObjectManager($this);
         $this->model = $objectHelper->getObject(
             SpecialPrice::class,
             [
                 'saleableItem' => $this->saleable,
                 'localeDate' => $this->localeDate,
-                'priceCurrency' => $this->priceCurrencyMock
+                'priceCurrency' => $this->priceCurrencyMock,
+                'specialPriceService' => $this->specialPriceService
             ]
         );
     }
@@ -103,6 +114,11 @@ public function testGetValue($regularPrice, $specialPrice, $isScopeDateInInterva
             ->with(WebsiteInterface::ADMIN_CODE, $specialFromDate, $specialToDate)
             ->willReturn($isScopeDateInInterval);
 
+        $this->specialPriceService->expects($this->once())
+            ->method('execute')
+            ->with($specialToDate)
+            ->willReturn($specialToDate);
+
         $this->priceCurrencyMock->expects($this->never())
             ->method('convertAndRound');