From 6999bdf57ee3fdef93fab5813f9567f9cb04fd9c Mon Sep 17 00:00:00 2001 From: Martin McKeaveney Date: Fri, 31 Jan 2025 16:58:29 +0000 Subject: [PATCH] add test for clearing results --- packages/server/src/jsRunner/tests/isolatedVM.spec.ts | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/packages/server/src/jsRunner/tests/isolatedVM.spec.ts b/packages/server/src/jsRunner/tests/isolatedVM.spec.ts index 5a9bc05d76..2cf97c8dd2 100644 --- a/packages/server/src/jsRunner/tests/isolatedVM.spec.ts +++ b/packages/server/src/jsRunner/tests/isolatedVM.spec.ts @@ -107,4 +107,14 @@ describe("Test isolated vm directly", () => { ) expect(result).toEqual([]) }) + + it("should ensure error results are cleared between runs", () => { + const context = {} + // throw error + const result = runJSWithIsolatedVM(`test.foo.bar = 123`, context) + expect(result).toEqual({}) + // ensure error not persisted across vms + const secondResult = runJSWithIsolatedVM(`return {}`, context) + expect(secondResult).toEqual({}) + }) })