MAGETWO-61039: Error when saving an existing product with an image

Sergey Shvets · Sergey Shvets · commit 0e3b41773624 · 2016-11-24T16:09:05.000+02:00
fixed unit-test
diff --git a/app/code/Magento/Catalog/Test/Unit/Ui/DataProvider/Product/Form/Modifier/ImagesTest.php b/app/code/Magento/Catalog/Test/Unit/Ui/DataProvider/Product/Form/Modifier/ImagesTest.php
@@ -17,14 +17,16 @@ class ImagesTest extends AbstractModifierTest
      */
     protected function createModel()
     {
-        $this->productMock->expects($this->once())->method('getId')->willReturn(2051);
-        $actualResult = $this->getModel()->modifyData($this->getSampleData());
-        $this->assertSame('', $actualResult[2051]['product']['media_gallery']['images'][0]['label']);
+        return $this->objectManager->getObject(Images::class, [
+            'locator' => $this->locatorMock,
+        ]);
     }
 
     public function testModifyData()
     {
-        $this->assertSame($this->getSampleData(), $this->getModel()->modifyData($this->getSampleData()));
+        $this->productMock->expects($this->once())->method('getId')->willReturn(2051);
+        $actualResult = $this->getModel()->modifyData($this->getSampleData());
+        $this->assertSame('', $actualResult[2051]['product']['media_gallery']['images'][0]['label']);
     }
 
     public function testModifyMeta()

Original file line number	Diff line number	Diff line change
`@@ -17,14 +17,16 @@ class ImagesTest extends AbstractModifierTest`
`17`	`17`	`*/`
`18`	`18`	`protected function createModel()`
`19`	`19`	`{`
`20`		`- $this->productMock->expects($this->once())->method('getId')->willReturn(2051);`
`21`		`- $actualResult = $this->getModel()->modifyData($this->getSampleData());`
`22`		`- $this->assertSame('', $actualResult[2051]['product']['media_gallery']['images'][0]['label']);`
	`20`	`+ return $this->objectManager->getObject(Images::class, [`
	`21`	`+ 'locator' => $this->locatorMock,`
	`22`	`+ ]);`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`public function testModifyData()`
`26`	`26`	`{`
`27`		`- $this->assertSame($this->getSampleData(), $this->getModel()->modifyData($this->getSampleData()));`
	`27`	`+ $this->productMock->expects($this->once())->method('getId')->willReturn(2051);`
	`28`	`+ $actualResult = $this->getModel()->modifyData($this->getSampleData());`
	`29`	`+ $this->assertSame('', $actualResult[2051]['product']['media_gallery']['images'][0]['label']);`
`28`	`30`	`}`
`29`	`31`
`30`	`32`	`public function testModifyMeta()`