fix(resource): configure the node http client depending on the protocol

Closes #35

Author: spirosikmd

README.md

@@ -42,7 +42,7 @@ See the code in the [example folder](https://github.com/usabilla/api-js-node/tre
 
 The client can be configured during instantiation with the following options:
 
-- `protocol` (default: `https`) - The protocol to use when making requests
+- `protocol` (default: `https`) - The protocol to use when making requests, this will also configure the type of node http client either `http` or `https`
 - `host` (default: `data.usabilla.com`) - The host to use when making requests
 - `port` (default: `null`) - The port to use when making requests
 - `iterator` (default: `true`) - Whether to iterate until all results are retrieved

src/resources/resource.js

@@ -1,4 +1,5 @@
 const https = require('https');
+const http = require('http');
 const packageJson = require('../../package.json');
 
 /**
@@ -12,6 +13,11 @@ class Resource {
     this.config = config;
     this.str = '';
     this.queryParams = {};
+    this.httpClient = this.getHttpClient(this.config.protocol);
+  }
+
+  getHttpClient(protocol) {
+    return protocol === 'http' ? http : https;
   }
 
   handleOnData(newJsonString) {
@@ -87,7 +93,7 @@ class Resource {
     };
 
     return new Promise((resolve, reject) => {
-      https
+      this.httpClient
         .get(requestOptions, this.handleGetResponse.bind(this, resolve, reject))
         .on('error', this.handleOnError.bind(this, reject));
     });

test/resources/resource.spec.js

@@ -26,6 +26,12 @@ describe('Resource', function() {
     );
   });
 
+  describe('getHttpClient', function() {
+    it('returns http when protocol is http', function() {
+      expect(this.resource.getHttpClient('http')).toBeDefined();
+    });
+  });
+
   describe('handleOnData', function() {
     it('concatenates new json string to existing json string', function() {
       this.resource.str = 'foo';