ACP2E-2213: Loading order through repository loads the order multiple times from DB

o-iegorov · o-iegorov · commit 533b1a2fe920 · 2023-08-22T14:18:30.000-05:00
diff --git a/app/code/Magento/Sales/Test/Unit/Model/OrderRepositoryTest.php b/app/code/Magento/Sales/Test/Unit/Model/OrderRepositoryTest.php
@@ -266,18 +266,6 @@ public function testGet()
     private function getOrderExtensionMock(): MockObject
     {
         $mockBuilder = $this->getMockBuilder(OrderExtensionInterface::class)->disableOriginalConstructor();
-        try {
-            $mockBuilder
-                ->onlyMethods([
-                        'setAppliedTaxes',
-                        'setConvertingFromQuote',
-                        'setItemAppliedTaxes',
-                        'setPaymentAdditionalInfo'
-                    ]);
-        } catch (RuntimeException $e) {
-            // Order extension already generated.
-        }
-
         return $mockBuilder->getMockForAbstractClass();
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -266,18 +266,6 @@ public function testGet()`
`266`	`266`	`private function getOrderExtensionMock(): MockObject`
`267`	`267`	`{`
`268`	`268`	`$mockBuilder = $this->getMockBuilder(OrderExtensionInterface::class)->disableOriginalConstructor();`
`269`		`- try {`
`270`		`- $mockBuilder`
`271`		`- ->onlyMethods([`
`272`		`- 'setAppliedTaxes',`
`273`		`- 'setConvertingFromQuote',`
`274`		`- 'setItemAppliedTaxes',`
`275`		`- 'setPaymentAdditionalInfo'`
`276`		`- ]);`
`277`		`- } catch (RuntimeException $e) {`
`278`		`- // Order extension already generated.`
`279`		`- }`
`280`		`-`
`281`	`269`	`return $mockBuilder->getMockForAbstractClass();`
`282`	`270`	`}`
`283`	`271`	`}`