Fix returning the result of a promise.

ltonetwork · Mar 8, 2023 · 5ae9f3f · 5ae9f3f
1 parent f6ae963
commit 5ae9f3f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/listener.ts b/src/listener.ts
@@ -11,8 +11,8 @@ export default class Listener {
         const source: WindowLike = (event.source as Window) || this.fallbackSource;
         const targetOrigin = event.origin && event.origin !== "null" ? event.origin : "*";
 
-        Promise.resolve(result).then(() => {
-            source.postMessage({'@rpc': RESPONSE_TYPE, channel, id, result}, targetOrigin);
+        Promise.resolve(result).then(r => {
+            source.postMessage({'@rpc': RESPONSE_TYPE, channel, id, result: r}, targetOrigin);
         });
     }
 

diff --git a/test/index.spec.ts b/test/index.spec.ts
@@ -19,7 +19,7 @@ describe("simple-iframe-rpc", () => {
         child = new JSDOM('').window;
         client = new Listener({
             add: (a, b) => a + b,
-            sub: (a, b) => a - b,
+            sub: (a, b) => Promise.resolve(a - b),
             err: () => { throw new Error("Oops"); }
         });
         client.listen(child, "*");
@@ -41,6 +41,11 @@ describe("simple-iframe-rpc", () => {
         assert.equal(result, 5);
     });
 
+    it("gives a result of a promise", async () => {
+        const result = await rpc.sub(3, 2);
+        assert.equal(result, 1);
+    });
+
     it("throws an error", async () => {
         try {
             await rpc.err();