Use POST method for REST transport (#66)

* use POST method for rest transports

* enable skipped test cases for `getInstalledProducts()`

Signed-off-by: Xu Meng <mengxumx@cn.ibm.com>

Author: dmabupt

lib/transports/irest.js

@@ -59,8 +59,12 @@ const iRestHttp = (config, xmlInput, done) => {
   const options = {
     host,
     port,
-    path: `${path}?${xmlEnc}`,
-    method: 'GET',
+    path,
+    method: 'POST',
+    headers: {
+      'Content-Type': 'application/x-www-form-urlencoded',
+      'Content-Length': Buffer.byteLength(xmlEnc)
+    }
   };
 
   const request = http.request(options, (response) => {
@@ -78,7 +82,7 @@ const iRestHttp = (config, xmlInput, done) => {
   request.on('error', (error) => {
     done(error, null);
   });
-
+  request.write(xmlEnc);
   request.end();
 };
 

test/functional/ToolkitFunctional.js

@@ -653,9 +653,6 @@ describe('Toolkit Functional Tests', () => {
     transports.forEach((transport) => {
       // eslint-disable-next-line func-names
       it(`returns info for installed products using ${transport.name} transport`, function (done) {
-        if (transport.name === 'rest') {
-          this.skip();
-        }
         const connection = transport.me;
 
         const toolkit = new Toolkit(connection);

test/functional/deprecated/iProdFunctional.js

@@ -129,9 +129,6 @@ describe('iProd Functional Tests', () => {
     transports.forEach((transport) => {
       // eslint-disable-next-line func-names
       it(`returns info for installed products using ${transport.name} transport`, function (done) {
-        if (transport.name === 'rest') {
-          this.skip();
-        }
         const connection = transport.me;
 
         const prod = new iProd(connection);