Browse Source

Fixing test case and removing console.log.

pull/1815/head
mike12345567 5 years ago
parent
commit
bfc4a7cc7b
  1. 1
      packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
  2. 5
      packages/server/__mocks__/node-fetch.js
  3. 5
      packages/server/src/automations/steps/outgoingWebhook.js
  4. 2
      packages/server/src/automations/tests/outgoingWebhook.spec.js

1
packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte

@ -14,7 +14,6 @@
$: allowDeleteTrigger = !steps.length
function deleteStep() {
console.log("Running")
automationStore.actions.deleteAutomationBlock(block)
}
</script>

5
packages/server/__mocks__/node-fetch.js

@ -4,6 +4,11 @@ module.exports = async (url, opts) => {
function json(body, status = 200) {
return {
status,
headers: {
get: () => {
return ["application/json"]
},
},
json: async () => {
return body
},

5
packages/server/src/automations/steps/outgoingWebhook.js

@ -77,7 +77,10 @@ module.exports.run = async function ({ inputs }) {
requestBody.length !== 0 &&
BODY_REQUESTS.indexOf(requestMethod) !== -1
) {
request.body = requestBody
request.body =
typeof requestBody === "string"
? requestBody
: JSON.stringify(requestBody)
request.headers = {
"Content-Type": "application/json",
}

2
packages/server/src/automations/tests/outgoingWebhook.spec.js

@ -25,7 +25,7 @@ describe("test the outgoing webhook action", () => {
expect(res.success).toEqual(true)
expect(res.response.url).toEqual("http://www.test.com")
expect(res.response.method).toEqual("POST")
expect(res.response.body.a).toEqual(1)
expect(JSON.parse(res.response.body).a).toEqual(1)
})
it("should return an error if something goes wrong in fetch", async () => {

Loading…
Cancel
Save