Handle request with undefined url

ctimmerm · Jul 24, 2020 · 78fe012 · 78fe012
1 parent a0026dc
commit 78fe012
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/src/handle_request.js b/src/handle_request.js
@@ -33,13 +33,13 @@ function makeResponse(result, config) {
 }
 
 function handleRequest(mockAdapter, resolve, reject, config) {
-  var url = config.url;
+  var url = config.url || "";
   // TODO we're not hitting this `if` in any of the tests, investigate
   if (
     config.baseURL &&
-    config.url.substr(0, config.baseURL.length) === config.baseURL
+    url.substr(0, config.baseURL.length) === config.baseURL
   ) {
-    url = config.url.slice(config.baseURL.length);
+    url = url.slice(config.baseURL.length);
   }
 
   delete config.adapter;
@@ -120,7 +120,7 @@ function handleRequest(mockAdapter, resolve, reject, config) {
       case "passthrough":
         mockAdapter.originalAdapter(config).then(resolve, reject);
         break;
-      case 'throwException':
+      case "throwException":
         throw utils.createCouldNotFindMockError(config);
       default:
         utils.settle(

diff --git a/test/pass_through.spec.js b/test/pass_through.spec.js
@@ -121,6 +121,14 @@ describe("passThrough tests (requires Node)", function () {
     });
   });
 
+  it("handle request with baseURL only", function () {
+    mock.onAny().passThrough();
+
+    return instance.get(undefined).then(function (response) {
+      expect(response.data).to.equal("/");
+    });
+  });
+
   it("handles request transformations properly", function () {
     mock.onGet("/foo").passThrough();