Merged PR 13: Fix azure-pipeline node kill
Some checks failed
continuous-integration/drone/push Build is failing

We use killall to stop node from running but az pipe catch SIGTERM.
Let's rather use SIGINT
This commit is contained in:
Eric Nguyen 2022-08-08 14:55:40 +00:00
parent 87369ee90d
commit 9dcfc5f226
2 changed files with 5 additions and 3 deletions

View file

@ -9,6 +9,7 @@ steps:
- node ./test-server/node-http.js &
- npm install
- npm run test:nowatch
- kill -2 %1 2>/dev/null
---
kind: pipeline
@ -20,4 +21,5 @@ steps:
commands:
- node ./test-server/node-http.js &
- npm install
- npm run test:nowatch
- npm run test:nowatch
- kill -2 %1 2>/dev/null

View file

@ -23,7 +23,7 @@ steps:
jobs
npm ci
npm run test:nowatch
killall node
kill -2 %1 2>/dev/null
displayName: 'Test on Node.js 16.x LTS'
- task: NodeTool@0
@ -37,5 +37,5 @@ steps:
jobs
npm ci
npm run test:nowatch
killall node
kill -2 %1 2>/dev/null
displayName: 'Test on Node.js 18.x Latest'