Skip to content

Commit 9cda5f0

Browse files
committed
788: Integration test rework. Shipment controller refactor.
1 parent 25ee332 commit 9cda5f0

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public function execute()
5353
{
5454
$this->shipmentLoader->setOrderId($this->getRequest()->getParam('order_id'));
5555
$this->shipmentLoader->setShipmentId($this->getRequest()->getParam('shipment_id'));
56-
$this->shipmentLoader->setShipment($this->shipmentProvider->getShipment());
56+
$this->shipmentLoader->setShipment($this->shipmentProvider->getShipmentData());
5757
$this->shipmentLoader->setTracking($this->getRequest()->getParam('tracking'));
5858
$shipment = $this->shipmentLoader->load();
5959
if ($shipment) {

app/code/Magento/Shipping/Test/Unit/Controller/Adminhtml/Order/Shipment/NewActionTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ protected function setUp()
135135
);
136136
$this->shipmentProviderMock = $this->getMockBuilder(\Magento\Shipping\Model\ShipmentProviderInterface::class)
137137
->disableOriginalConstructor()
138-
->setMethods(['getShipment'])
138+
->setMethods(['getShipmentData'])
139139
->getMockForAbstractClass();
140140
$this->actionFlag = $this->createPartialMock(\Magento\Framework\App\ActionFlag::class, ['get']);
141141
$this->helper = $this->createPartialMock(\Magento\Backend\Helper\Data::class, ['getUrl']);
@@ -270,7 +270,7 @@ public function testExecute()
270270
->with('menu')
271271
->will($this->returnValue($menuBlock));
272272
$this->shipmentProviderMock->expects($this->once())
273-
->method('getShipment')
273+
->method('getShipmentData')
274274
->willReturn($shipmentData);
275275

276276
$this->assertNull($this->newAction->execute());

0 commit comments

Comments
 (0)