fix: swap host for base_url since it is not a host

Author: chribben

README.md

@@ -4,5 +4,5 @@ Tests a node server firing generated requests (using its Swagger spec) at it
 ## Usage
 ```javascript
 import { testServer } from '@springworks/server-tester';
-testServer('http://locahost', 3001, swagger_spec);
+testServer({ base_url: 'http://locahost', port: 3001, swagger_spec });
 ```

src/index.js

@@ -2,15 +2,15 @@ import { constructRequests } from '@springworks/request-baker';
 import rp from 'request-promise';
 import path from 'path';
 
-export async function testServer({ host, port, swagger_spec, generateRequests }) {
+export async function testServer({ base_url, port, swagger_spec, generateRequests }) {
   const getRequests = generateRequests || constructRequests;
   const { requests } = await getRequests(swagger_spec);
   for (const generated_request of requests) {
     const { method, body: json, qs, headers, base_path } = generated_request;
     const request_path = path.join(base_path || '', generated_request.path);
-    const base_url = `${host.replace(/\/+$/, '')}${port ? `:${port}` : ''}`;
+    const base_url_with_port = `${base_url.replace(/\/+$/, '')}${port ? `:${port}` : ''}`;
     try {
-      await rp({ uri: request_path, baseUrl: base_url, method, qs, headers, json });
+      await rp({ uri: request_path, baseUrl: base_url_with_port, method, qs, headers, json });
     }
     catch (err) {
       const message = `

test/acceptance/test-server-test.js

@@ -38,7 +38,7 @@ describe('test/acceptance/test-server-test.js', () => {
       });
 
       it('should not err', () => {
-        return testServer({ host: 'http://localhost', port: 3001, swagger_spec }).then(res => {
+        return testServer({ base_url: 'http://localhost', port: 3001, swagger_spec }).then(res => {
         });
       });
 

test/unit/test-server-test.js

@@ -55,7 +55,7 @@ describe('test/unit/test-server-test.js', () => {
 
       describe('without base path', () => {
         beforeEach('stub rp', () => {
-          return testServer({ host: 'http://localhost/', port: 3001, swagger_spec: api });
+          return testServer({ base_url: 'http://localhost/', port: 3001, swagger_spec: api });
         });
 
         it('should invoke a request once with correct params', () => {
@@ -84,7 +84,7 @@ describe('test/unit/test-server-test.js', () => {
         });
 
         beforeEach('stub rp', () => {
-          return testServer({ host: 'http://localhost/', port: 3001, swagger_spec: api });
+          return testServer({ base_url: 'http://localhost/', port: 3001, swagger_spec: api });
         });
 
         it('should invoke a request once with correct params', () => {
@@ -107,7 +107,7 @@ describe('test/unit/test-server-test.js', () => {
 
       describe('excluding port', () => {
         beforeEach('stub rp', () => {
-          return testServer({ host: 'http://localhost/', swagger_spec: api });
+          return testServer({ base_url: 'http://localhost/', swagger_spec: api });
         });
 
         it('should invoke a request once with correct params', () => {