chore: skip non-deterministic tests during recording

aws · Oct 6, 2022 · 2a5b889 · 2a5b889
1 parent 77d3f14
commit 2a5b889
Showing 1 changed file with 43 additions and 33 deletions.
diff --git a/packages/@jsii/kernel/src/kernel.test.ts b/packages/@jsii/kernel/src/kernel.test.ts
@@ -78,6 +78,10 @@ defineTest.skip = function (
   return defineTest(name, method, test.skip);
 };
 
+defineTest.skipIf = function (expr: boolean): typeof defineTest.skip {
+  return expr ? defineTest.skip : defineTest;
+};
+
 // Note: this test asserts file permissions, which work differently on Windows, so we skip it there
 (process.platform === 'win32' ? test.skip : test)(
   'load preserves file permissions',
@@ -2138,42 +2142,48 @@ defineTest('Override transitive property', (sandbox) => {
   expect(propValue).toBe('N3W');
 });
 
-defineTest('getBinScriptCommand() returns output', (sandbox) => {
-  const result = sandbox.getBinScriptCommand({
-    assembly: 'jsii-calc',
-    script: 'calc',
-  });
+defineTest.skipIf(!!recordingOutput)(
+  'getBinScriptCommand() returns output',
+  (sandbox) => {
+    const result = sandbox.getBinScriptCommand({
+      assembly: 'jsii-calc',
+      script: 'calc',
+    });
 
-  expect(result).toMatchObject<api.GetScriptCommandResponse>({
-    command: expect.stringMatching(
-      /node_modules[/\\]jsii-calc[/\\]bin[/\\]run$/,
-    ),
-    args: [],
-    env: {
-      NODE_OPTIONS: expect.any(String),
-      PATH: expect.any(String),
-    },
-  });
-});
+    expect(result).toMatchObject<api.GetScriptCommandResponse>({
+      command: expect.stringMatching(
+        /node_modules[/\\]jsii-calc[/\\]bin[/\\]run$/,
+      ),
+      args: [],
+      env: {
+        NODE_OPTIONS: expect.any(String),
+        PATH: expect.any(String),
+      },
+    });
+  },
+);
 
-defineTest('getBinScriptCommand() accepts arguments', (sandbox) => {
-  const result = sandbox.getBinScriptCommand({
-    assembly: 'jsii-calc',
-    script: 'calc',
-    args: ['arg1', 'arg2'],
-  });
+defineTest.skipIf(!!recordingOutput)(
+  'getBinScriptCommand() accepts arguments',
+  (sandbox) => {
+    const result = sandbox.getBinScriptCommand({
+      assembly: 'jsii-calc',
+      script: 'calc',
+      args: ['arg1', 'arg2'],
+    });
 
-  expect(result).toMatchObject<api.GetScriptCommandResponse>({
-    command: expect.stringMatching(
-      /node_modules[/\\]jsii-calc[/\\]bin[/\\]run$/,
-    ),
-    args: ['arg1', 'arg2'],
-    env: {
-      NODE_OPTIONS: expect.any(String),
-      PATH: expect.any(String),
-    },
-  });
-});
+    expect(result).toMatchObject<api.GetScriptCommandResponse>({
+      command: expect.stringMatching(
+        /node_modules[/\\]jsii-calc[/\\]bin[/\\]run$/,
+      ),
+      args: ['arg1', 'arg2'],
+      env: {
+        NODE_OPTIONS: expect.any(String),
+        PATH: expect.any(String),
+      },
+    });
+  },
+);
 
 defineTest('invokeBinScript() return output', (sandbox) => {
   const result = sandbox.invokeBinScript({