process code review comments

Author: AntonEvers

app/code/Magento/Persistent/Test/Unit/Model/QuoteManagerTest.php

@@ -230,7 +230,6 @@ public function testSetGuest()
         $this->quoteMock->expects($this->once())->method('getItemsQty')->willReturn(1);
         $extensionAttributes = $this->getMockBuilder(CartExtensionInterface::class)
             ->addMethods(['getShippingAssignments', 'setShippingAssignments'])
-            ->disableArgumentCloning()
             ->getMockForAbstractClass();
         $shippingAssignment = $this->createMock(ShippingAssignmentInterface::class);
         $extensionAttributes->expects($this->once())

app/code/Magento/Quote/Test/Unit/Model/Quote/Address/Total/SubtotalTest.php

@@ -112,9 +112,6 @@ public function testCollect($price, $originalPrice, $itemHasParent, $expectedPri
         /** @var Address|MockObject $address */
         $address = $this->getMockBuilder(Address::class)
             ->disableOriginalConstructor()
-            ->disableOriginalClone()
-            ->disableArgumentCloning()
-            ->disallowMockingUnknownTypes()
             ->onlyMethods(['removeItem', 'getQuote'])
             ->addMethods(['setTotalQty', 'getTotalQty'])
             ->getMock();
@@ -167,9 +164,6 @@ public function testCollect($price, $originalPrice, $itemHasParent, $expectedPri
 
         $total = $this->getMockBuilder(Total::class)
             ->disableOriginalConstructor()
-            ->disableOriginalClone()
-            ->disableArgumentCloning()
-            ->disallowMockingUnknownTypes()
             ->addMethods(['setVirtualAmount', 'setBaseVirtualAmount'])
             ->getMock();
         $total->expects($this->once())->method('setBaseVirtualAmount')->willReturnSelf();
@@ -194,7 +188,6 @@ public function testFetch()
         $quoteMock = $this->createMock(Quote::class);
         $totalMock = $this->getMockBuilder(Total::class)
             ->addMethods(['getSubtotal'])
-            ->disableArgumentCloning()
             ->getMockForAbstractClass();
         $totalMock->expects($this->once())->method('getSubtotal')->willReturn(100);
 
@@ -241,9 +234,6 @@ public function testCollectWithInvalidItems()
             ->with($addressItemId);
         $addressItem = $this->getMockBuilder(AddressItem::class)
             ->disableOriginalConstructor()
-            ->disableOriginalClone()
-            ->disableArgumentCloning()
-            ->disallowMockingUnknownTypes()
             ->onlyMethods(['getId'])
             ->addMethods(['getQuoteItemId'])
             ->getMock();

app/code/Magento/Sales/Test/Unit/Model/Order/CreditmemoFactoryTest.php

@@ -51,9 +51,6 @@ protected function setUp(): void
     {
         $this->orderItemMock = $this->getMockBuilder(Item::class)
             ->disableOriginalConstructor()
-            ->disableOriginalClone()
-            ->disableArgumentCloning()
-            ->disallowMockingUnknownTypes()
             ->onlyMethods(['getChildrenItems', 'isDummy', 'getId', 'getParentItemId'])
             ->addMethods(['getHasChildren'])
             ->getMock();

lib/internal/Magento/Framework/Pricing/Test/Unit/Adjustment/CollectionTest.php

@@ -71,7 +71,6 @@ function ($code) use ($adjustmentsData) {
     /**
      * @param string[] $adjustments
      * @param string[] $expectedResult
-     *
      * @dataProvider getItemsDataProvider
      */
     public function testGetItems($adjustments, $expectedResult)
@@ -92,15 +91,14 @@ public function getItemsDataProvider()
             [['adj1'], ['adj1']],
             [['adj4'], ['adj4']],
             [['adj1', 'adj4'], ['adj1', 'adj4']],
-            [['adj1', 'adj2', 'adj3', 'adj4'], ['adj3', 'adj1', 'adj2', 'adj4']],
+            [['adj1', 'adj2', 'adj3', 'adj4'], ['adj3', 'adj1', 'adj2', 'adj4']]
         ];
     }
 
     /**
      * @param string[] $adjustments
      * @param string $code
      * @param $expectedResult
-     *
      * @dataProvider getItemByCodeDataProvider
      */
     public function testGetItemByCode($adjustments, $code, $expectedResult)