pr comments

packages/server/src/automations/tests/scenarios/looping.spec.ts

@@ -299,9 +299,7 @@ describe("Loop automations", () => {
       { name: "Row 3", value: 3, tableId: table._id },
     ]
 
-    for (const row of rows) {
-      await config.createRow(row)
-    }
+    await config.api.row.bulkImport(table._id!, { rows })
 
     const builder = createAutomationBuilder({
       name: "Test Loop and Update Row",
@@ -385,9 +383,7 @@ describe("Loop automations", () => {
       { name: "Row 3", value: 3, tableId: table._id },
     ]
 
-    for (const row of rows) {
-      await config.createRow(row)
-    }
+    await config.api.row.bulkImport(table._id!, { rows })
 
     const builder = createAutomationBuilder({
       name: "Test Loop and Delete Row",

packages/server/src/automations/tests/scenarios/scenarios.spec.ts

@@ -444,43 +444,48 @@ describe("Automation Scenarios", () => {
       },
     ]
 
-    testCases.forEach(({ condition, value, rowValue, expectPass }) => {
-      it(`should ${
-        expectPass ? "pass" : "fail"
-      } the filter when condition is "${condition}" and value is ${value}`, async () => {
-        const builder = createAutomationBuilder({
-          name: `Test ${condition}`,
+    it.each(testCases)(
+      "should pass the filter when condition is %s",
+      async ({ condition, value, rowValue, expectPass }) => {
+        it(`should ${
+          expectPass ? "pass" : "fail"
+        } the filter when condition is "${condition}" and value is ${value}`, async () => {
+          const builder = createAutomationBuilder({
+            name: `Test ${condition}`,
+          })
+
+          const results = await builder
+            .appAction({ fields: {} })
+            .createRow({
+              row: {
+                name: `${condition} Test`,
+                value: rowValue,
+                tableId: table._id,
+              },
+            })
+            .queryRows({
+              tableId: table._id!,
+            })
+            .filter({
+              field: "{{ steps.2.rows.0.value }}",
+              condition,
+              value,
+            })
+            .serverLog({
+              text: `${condition} condition ${
+                expectPass ? "passed" : "failed"
+              }`,
+            })
+            .run()
+
+          expect(results.steps[2].outputs.result).toBe(expectPass)
+          if (expectPass) {
+            expect(results.steps[3].outputs.success).toBeTrue()
+          } else {
+            expect(results.steps[3]).toBeUndefined()
+          }
         })
-
-        const results = await builder
-          .appAction({ fields: {} })
-          .createRow({
-            row: {
-              name: `${condition} Test`,
-              value: rowValue,
-              tableId: table._id,
-            },
-          })
-          .queryRows({
-            tableId: table._id!,
-          })
-          .filter({
-            field: "{{ steps.2.rows.0.value }}",
-            condition,
-            value,
-          })
-          .serverLog({
-            text: `${condition} condition ${expectPass ? "passed" : "failed"}`,
-          })
-          .run()
-
-        expect(results.steps[2].outputs.result).toBe(expectPass)
-        if (expectPass) {
-          expect(results.steps[3].outputs.success).toBeTrue()
-        } else {
-          expect(results.steps[3]).toBeUndefined()
-        }
-      })
-    })
+      }
+    )
   })
 })