Fix background() missing return option support

hapijs · Oct 23, 2024 · b751478 · b751478
1 parent f6c3b19
commit b751478
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/index.js b/lib/index.js
@@ -78,7 +78,7 @@ exports.background = async function (operation, action = 'rethrow', types = 'sys
         }
     }
     catch (err) {
-        exports[action](err, types, options);
+        return exports[action](err, types, options);
     }
 };
 

diff --git a/test/index.js b/test/index.js
@@ -427,6 +427,19 @@ describe('Bounce', () => {
 
             Bounce.background(test, 'rethrow', 'system');
         });
+
+        it('supports the return option', async () => {
+
+            const test = async () => {
+
+                await Hoek.wait(1);
+                throw new SyntaxError('Something');
+            };
+
+            const res = await Bounce.background(test(), 'rethrow', 'system', { return: true });
+            expect(res).to.exist();
+            expect(res).to.be.an.error(SyntaxError);
+        });
     });
 
     describe('isBoom()', () => {