#23386: Copy Service does not works properly for Entities which extends Data Object and implements ExtensibleDataInterface.

Reformat code.

Author: al.kravchuk

lib/internal/Magento/Framework/DataObject/Test/Unit/CopyTest.php

@@ -8,6 +8,9 @@
 
 use Magento\Quote\Model\Quote\Address;
 
+/**
+ * Unit tests coverage for @see \Magento\Framework\DataObject\Copy
+ */
 class CopyTest extends \PHPUnit\Framework\TestCase
 {
     /**
@@ -91,7 +94,7 @@ public function testCopyFieldsetToTargetWhenFieldExists()
         $data = [
             'target' => new \Magento\Framework\DataObject([$this->targetMock]),
             'source' => $this->sourceMock,
-            'root'   => 'global',
+            'root' => 'global',
         ];
         $this->eventManagerMock->expects($this->once())->method('dispatch')->with($eventName, $data);
         $this->assertEquals(
@@ -124,7 +127,7 @@ public function testCopyFieldsetToTargetWhenTargetNotArray()
         $data = [
             'target' => $this->targetMock,
             'source' => $this->sourceMock,
-            'root'   => 'global',
+            'root' => 'global',
         ];
         $this->eventManagerMock->expects($this->once())->method('dispatch')->with($eventName, $data);
         $this->assertEquals(
@@ -160,7 +163,7 @@ public function testGetCopyFieldsetToTargetWhenTargetIsArray()
         $data = [
             'target' => new \Magento\Framework\DataObject($newTarget),
             'source' => $this->sourceMock,
-            'root'   => 'global',
+            'root' => 'global',
         ];
         $this->eventManagerMock->expects($this->once())->method('dispatch')->with($eventName, $data);
         $this->assertEquals(
@@ -178,14 +181,21 @@ public function testGetCopyFieldsetToTargetWhenTargetIsExtensibleDataInterface()
             ->with('fieldset', 'global')
             ->will($this->returnValue($fields));
 
-        $sourceMock = $this->createPartialMock(\Magento\Framework\Api\ExtensibleDataInterface::class, [
-                'getExtensionAttributes', 'getCode'
-            ]);
-        $targetMock = $this->createPartialMock(\Magento\Framework\Api\ExtensibleDataInterface::class, [
+        $sourceMock = $this->createPartialMock(
+            \Magento\Framework\Api\ExtensibleDataInterface::class,
+            [
+                'getExtensionAttributes',
+                'getCode'
+            ]
+        );
+        $targetMock = $this->createPartialMock(
+            \Magento\Framework\Api\ExtensibleDataInterface::class,
+            [
                 'getExtensionAttributes',
                 'setCode',
                 'setExtensionAttributes'
-            ]);
+            ]
+        );
 
         $sourceMock
             ->expects($this->any())
@@ -224,12 +234,18 @@ public function testGetCopyFieldsetToTargetWhenTargetIsAbstractSimpleObject()
             ->with('fieldset', 'global')
             ->will($this->returnValue($fields));
 
-        $sourceMock = $this->createPartialMock(\Magento\Framework\Api\AbstractSimpleObject::class, [
+        $sourceMock = $this->createPartialMock(
+            \Magento\Framework\Api\AbstractSimpleObject::class,
+            [
                 '__toArray'
-            ]);
-        $targetMock = $this->createPartialMock(\Magento\Framework\Api\AbstractSimpleObject::class, [
+            ]
+        );
+        $targetMock = $this->createPartialMock(
+            \Magento\Framework\Api\AbstractSimpleObject::class,
+            [
                 'setData'
-            ]);
+            ]
+        );
 
         $sourceMock
             ->expects($this->once())
@@ -312,15 +328,21 @@ public function testGetExtensionAttributeForDataObjectChild()
             ->with('fieldset', 'global')
             ->will($this->returnValue($fields));
 
-        $sourceMock = $this->createPartialMock(Address::class, [
-            'getExtensionAttributes', 'getCode'
-        ]);
+        $sourceMock = $this->createPartialMock(
+            Address::class,
+            [
+                'getExtensionAttributes',
+                'getCode'
+            ]
+        );
         $targetMock = $this->createPartialMock(
-            Address::class, [
-            'getExtensionAttributes',
-            'setCode',
-            'setExtensionAttributes'
-        ]);
+            Address::class,
+            [
+                'getExtensionAttributes',
+                'setCode',
+                'setExtensionAttributes'
+            ]
+        );
 
         $sourceMock
             ->expects($this->any())
@@ -349,7 +371,7 @@ public function testGetExtensionAttributeForDataObjectChild()
         $this->assertEquals($result, $targetMock);
     }
 
-    public function testGetDataObjectFieldFromExtensbielEntity()
+    public function testGetDataObjectFieldFromExtensibleEntity()
     {
         $fields['code']['aspect'] = '*';
         $this->fieldsetConfigMock
@@ -358,13 +380,18 @@ public function testGetDataObjectFieldFromExtensbielEntity()
             ->with('fieldset', 'global')
             ->will($this->returnValue($fields));
 
-        $sourceMock = $this->createPartialMock(Address::class, [
-            'getExtensionAttributes'
-        ]);
+        $sourceMock = $this->createPartialMock(
+            Address::class,
+            [
+                'getExtensionAttributes'
+            ]
+        );
         $targetMock = $this->createPartialMock(
-            Address::class, [
-            'getExtensionAttributes'
-        ]);
+            Address::class,
+            [
+                'getExtensionAttributes'
+            ]
+        );
 
         $sourceMock
             ->expects($this->any())
@@ -378,7 +405,6 @@ public function testGetDataObjectFieldFromExtensbielEntity()
             ->method('getExtensionAttributes')
             ->willReturn(null);
 
-
         $this->eventManagerMock->expects($this->once())->method('dispatch');
         $result = $this->copy->copyFieldsetToTarget('fieldset', 'aspect', $sourceMock, $targetMock);
         $this->assertEquals($result, $targetMock);