Merge branch 'MAGETWO-50026' into bugfixes

Author: olysenko

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/CreditmemoGetTest.php

@@ -105,5 +105,10 @@ public function testCreditmemoGet()
             $this->assertArrayHasKey($field, $actual);
             $this->assertEquals($expected[$field], $actual[$field]);
         }
+
+        //check that nullable fields were marked as optional and were not sent
+        foreach ($actual as $value) {
+            $this->assertNotNull($value);
+        }
     }
 }

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/InvoiceGetTest.php

@@ -48,5 +48,10 @@ public function testInvoiceGet()
             $this->assertArrayHasKey($field, $result);
             $this->assertEquals($value, $result[$field]);
         }
+
+        //check that nullable fields were marked as optional and were not sent
+        foreach ($result as $value) {
+            $this->assertNotNull($value);
+        }
     }
 }

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/OrderGetTest.php

@@ -81,5 +81,10 @@ public function testOrderGet()
         foreach ($expectedBillingAddressNotEmpty as $field) {
             $this->assertArrayHasKey($field, $result['billing_address']);
         }
+
+        //check that nullable fields were marked as optional and were not sent
+        foreach ($result as $value) {
+            $this->assertNotNull($value);
+        }
     }
 }

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/OrderItemGetTest.php

@@ -53,6 +53,14 @@ public function testGet()
 
         $this->assertTrue(is_array($response));
         $this->assertOrderItem($orderItem, $response);
+
+        //check that nullable fields were marked as optional and were not sent
+        foreach ($response as $fieldName => $value) {
+            if ($fieldName == 'sku') {
+                continue;
+            }
+            $this->assertNotNull($value);
+        }
     }
 
     /**