@budibase/server 0.9.125-alpha.13 → 0.9.125-alpha.17

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@budibase/server",
   "email": "hi@budibase.com",
-  "version": "0.9.125-alpha.13",
+  "version": "0.9.125-alpha.17",
   "description": "Budibase Web Server",
   "main": "src/index.js",
   "repository": {
@@ -64,9 +64,9 @@
   "author": "Budibase",
   "license": "AGPL-3.0-or-later",
   "dependencies": {
-    "@budibase/auth": "^0.9.125-alpha.13",
-    "@budibase/client": "^0.9.125-alpha.13",
-    "@budibase/string-templates": "^0.9.125-alpha.13",
+    "@budibase/auth": "^0.9.125-alpha.17",
+    "@budibase/client": "^0.9.125-alpha.17",
+    "@budibase/string-templates": "^0.9.125-alpha.17",
     "@elastic/elasticsearch": "7.10.0",
     "@koa/router": "8.0.0",
     "@sendgrid/mail": "7.1.1",
@@ -142,5 +142,5 @@
     "typescript": "^4.3.5",
     "update-dotenv": "^1.1.1"
   },
-  "gitHead": "bede089cbecbf068aee9ec4861da5f0e2887fa35"
+  "gitHead": "70aad3b9fe19b04284ec1f210a55998b37618ee8"
 }

package/src/api/controllers/automation.js

@@ -189,15 +189,27 @@ exports.trigger = async function (ctx) {
   }
 }
 
+function prepareTestInput(input) {
+  // prepare the test parameters
+  if (input.id && input.row) {
+    input.row._id = input.id
+  }
+  if (input.revision && input.row) {
+    input.row._rev = input.revision
+  }
+  return input
+}
+
 exports.test = async function (ctx) {
   const appId = ctx.appId
   const db = new CouchDB(appId)
   let automation = await db.get(ctx.params.id)
   await setTestFlag(automation._id)
+  const testInput = prepareTestInput(ctx.request.body)
   const response = await triggers.externalTrigger(
     automation,
     {
-      ...ctx.request.body,
+      ...testInput,
       appId,
     },
     { getResponses: true }

package/src/api/routes/tests/metadata.spec.js

@@ -62,4 +62,4 @@ describe("/metadata", () => {
       expect(metadata.test).toBeUndefined()
     })
   })
-})
+})

package/src/automations/steps/outgoingWebhook.js

@@ -97,12 +97,16 @@ exports.run = async function ({ inputs }) {
       "Content-Type": "application/json",
     }
 
-    if (headers && headers.length !== 0) {
+    if (headers) {
       try {
-        const customHeaders = JSON.parse(headers)
+        const customHeaders =
+          typeof headers === "string" ? JSON.parse(headers) : headers
         request.headers = { ...request.headers, ...customHeaders }
       } catch (err) {
-        console.error(err)
+        return {
+          success: false,
+          response: "Unable to process headers, must be a JSON object.",
+        }
       }
     }
   }

package/src/automations/tests/automation.spec.js

@@ -83,4 +83,4 @@ describe("Run through some parts of the automations system", () => {
     expect(output.d).toBe(1)
     expect(output.e).toBe("help")
   })
-})
+})

package/src/automations/tests/delay.spec.js

@@ -9,4 +9,4 @@ describe("test the delay logic", () => {
     // divide by two just so that test will always pass as long as there was some sort of delay
     expect(now - before).toBeGreaterThanOrEqual(time / 2)
   })
-})
+})

package/src/automations/tests/filter.spec.js

@@ -45,4 +45,4 @@ describe("test the filter logic", () => {
   it("check objects always false", async () => {
     await checkFilter({}, FilterConditions.EQUAL, {}, false)
   })
-})
+})

package/src/automations/tests/queryRows.spec.js

@@ -48,4 +48,4 @@ describe("Test a query step automation", () => {
     expect(res.rows.length).toBe(2)
     expect(res.rows[0].name).toBe(NAME)
   })
-})
+})