Add support for skipping response parsing for operations with no defi…

…ned response
nuxwin · Jan 19, 2016 · 66db060 · 66db060
1 parent 0ca3101
commit 66db060
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/src/Api/Parser/AbstractRestParser.php b/src/Api/Parser/AbstractRestParser.php
@@ -53,7 +53,10 @@ public function __invoke(
             }
         }
 
-        if (!$payload && $response->getBody()->getSize() > 0) {
+        if (!$payload
+            && $response->getBody()->getSize() > 0
+            && count($output->getMembers()) > 0
+        ) {
             // if no payload was found, then parse the contents of the body
             $this->payload($response, $output, $result);
         }

diff --git a/tests/Api/test_cases/protocols/output/rest-json.json b/tests/Api/test_cases/protocols/output/rest-json.json
@@ -447,6 +447,26 @@
       }
     ]
   },
+  {
+    "description": "Ignores undefined output",
+    "metadata": {
+      "protocol": "rest-json"
+    },
+    "shapes": {},
+    "cases": [
+      {
+        "given": {
+          "name": "OperationName"
+        },
+        "result": {},
+        "response": {
+          "status_code": 200,
+          "headers": {},
+          "body": "OK"
+        }
+      }
+    ]
+  },
   {
     "description": "Supports header maps",
     "metadata": {