MAGETWO-54677: Order status change after Shipment creation through API

Author: valdislav

app/code/Magento/Sales/Test/Unit/Model/Order/Shipment/Validation/QuantityValidatorTest.php

@@ -45,7 +45,10 @@ public function testValidateTrackWithoutOrderId()
         $this->shipmentMock->expects($this->once())
             ->method('getOrderId')
             ->willReturn(null);
-        $this->assertEquals([__('Order Id is required for shipment document')], $this->validator->validate($this->shipmentMock));
+        $this->assertEquals(
+            [__('Order Id is required for shipment document')],
+            $this->validator->validate($this->shipmentMock)
+        );
     }
 
     public function testValidateTrackWithoutItems()
@@ -56,6 +59,9 @@ public function testValidateTrackWithoutItems()
         $this->shipmentMock->expects($this->once())
             ->method('getItems')
             ->willReturn(null);
-        $this->assertEquals([__('You can\'t create a shipment without products.')], $this->validator->validate($this->shipmentMock));
+        $this->assertEquals(
+            [__('You can\'t create a shipment without products.')],
+            $this->validator->validate($this->shipmentMock)
+        );
     }
 }

app/code/Magento/Shipping/Controller/Adminhtml/Order/Shipment/Save.php

@@ -7,7 +7,6 @@
 namespace Magento\Shipping\Controller\Adminhtml\Order\Shipment;
 
 use Magento\Backend\App\Action;
-use Magento\Framework\App\ObjectManager;
 use Magento\Sales\Model\Order\Shipment\Validation\QuantityValidator;
 
 /**
@@ -193,7 +192,7 @@ public function execute()
     private function getShipmentValidator()
     {
         if ($this->shipmentValidator === null) {
-            $this->shipmentValidator = ObjectManager::getInstance()->get(
+            $this->shipmentValidator = $this->_objectManager->get(
                 \Magento\Sales\Model\Order\Shipment\ShipmentValidatorInterface::class
             );
         }