Fix Delete composition

src/Mutate.test.tsx

@@ -78,8 +78,8 @@ describe("Mutate", () => {
       expect(children.mock.calls[2][1].loading).toEqual(false);
     });
 
-    it("should call the correct url with a specific id (with base in Mutate)", async () => {
-      nock("https://my-awesome-api.fake")
+    it("should call the correct url with a specific id (with base and path in Mutate)", async () => {
+      nock("https://my-awesome-api.fake/user")
         .delete("/plop")
         .reply(200, { id: 1 });
 
@@ -89,7 +89,7 @@ describe("Mutate", () => {
       // setup - first render
       render(
         <RestfulProvider base="https://my-awesome-api.fake">
-          <Mutate verb="DELETE" path="user">
+          <Mutate verb="DELETE" base="https://my-awesome-api.fake" path="user">
             {children}
           </Mutate>
         </RestfulProvider>,

src/Mutate.tsx

@@ -1,7 +1,7 @@
 import * as React from "react";
 import RestfulReactProvider, { InjectedProps, RestfulReactConsumer, RestfulReactProviderProps } from "./Context";
 import { GetState } from "./Get";
-import { composePath, composePathWithBody, composeUrl } from "./util/composeUrl";
+import { composePath, composeUrl } from "./util/composeUrl";
 import { processResponse } from "./util/processResponse";
 
 /**
@@ -145,11 +145,11 @@ class ContextlessMutate<TData, TError> extends React.Component<
     const makeRequestPath = () => {
       if (__internal_hasExplicitBase) {
         return verb === "DELETE" && typeof body === "string"
-          ? composeUrl(base!, "", composePathWithBody(path!, body))
+          ? composeUrl(base!, "", composePath(path!, body))
           : composeUrl(base!, "", path || "");
       } else {
         return verb === "DELETE" && typeof body === "string"
-          ? composeUrl(base!, parentPath!, composePathWithBody(path!, body))
+          ? composeUrl(base!, parentPath!, composePath(path!, body))
           : composeUrl(base!, parentPath!, path!);
       }
     };

src/util/composeUrl.test.ts

@@ -1,4 +1,4 @@
-import { composePath, composePathWithBody, composeUrl } from "./composeUrl";
+import { composePath, composeUrl } from "./composeUrl";
 
 describe("compose paths and urls", () => {
   it("should handle empty parentPath with absolute path", () => {
@@ -75,19 +75,3 @@ describe("compose paths and urls", () => {
     );
   });
 });
-
-describe("compose path with body", () => {
-  it("should compose body with absolute path", () => {
-    const path = "/absolute";
-    const body = "?somebody";
-
-    expect(composePathWithBody(path, body)).toBe("/absolute?somebody");
-  });
-
-  it("should compose body with relative path", () => {
-    const path = "relative";
-    const body = "?somebody";
-
-    expect(composePathWithBody(path, body)).toBe("relative?somebody");
-  });
-});

src/util/composeUrl.ts

@@ -23,5 +23,3 @@ export const composePath = (parentPath: string = "", path: string = ""): string
     return parentPath;
   }
 };
-
-export const composePathWithBody = (path: string, body: string): string => url.resolve(path, body);