fix: use object for error

open-rpc · May 21, 2019 · efd0f61 · efd0f61
1 parent 2d91e5a
commit efd0f61
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/src/RequestManager.test.ts b/src/RequestManager.test.ts
@@ -46,7 +46,7 @@ describe("client-js", () => {
       transport.connection.on("message", () => {
         fn(JSON.stringify({
           jsonrpc: "2.0",
-          id: 1,
+          id: 3,
           error: {
             code: 0,
             message: "out of order",
@@ -58,7 +58,13 @@ describe("client-js", () => {
       });
     };
     c.connect();
-    expect(c.request("foo", [])).rejects.toThrow();
+    expect(c.request("foo", [])).rejects.toBe({
+      code: 0,
+      message: "out of order",
+      data: {
+        foo: "bar",
+      },
+    });
   });
 
 });
diff --git a/src/RequestManager.ts b/src/RequestManager.ts
@@ -59,15 +59,9 @@ class RequestManager {
     // call request callback for id
     if (req) {
       if (parsedData.error) {
-        req.reject(new Error(
-          [
-            `code: ${parsedData.error.code}`,
-            `message: ${parsedData.error.message}`,
-            `data: ${JSON.stringify(parsedData.error.data)}`,
-          ].join("\n"),
-        ));
+        req.reject(parsedData.error);
       } else {
-        req.resolve(parsedData);
+        req.resolve(parsedData.result);
       }
       delete this.requests[parsedData.id];
     }