Fix(utils.isNode) Incompatibility of utils.isNode Function with signal-exit #1889

[vishal6557]

Fix #1889

Thank you for the time you are putting into AlaSQL!

[mathiasrw]

Love it. Thanks!

[Warfront1]

Nice job @vishal6557 and @mathiasrw. 👍

My tests are now passing. Dockerfile:

FROM node:16
WORKDIR /test
RUN npm init -y
ARG ALASQL_VERSION=4.2.7
RUN npm install alasql@$ALASQL_VERSION signal-exit@3.0.7
# Create the Node.js script inline
RUN echo "const onExit = require('signal-exit');\n\
onExit(() => console.log('exit'));\n\
const alasql = require('alasql');\n\
const utils = alasql.utils;\n\
console.log('The current value of utils.isNode is: ', utils.isNode);\n\
" > utilsNodeCheck.js
# Run the Node.js script
CMD ["node", "utilsNodeCheck.js"]

CMD:

docker build --build-arg ALASQL_VERSION=4.2.7 -t my-image:4.2.7 . && docker run my-image:4.2.7

Output: The current value of utils.isNode is: false

CMD:

docker build --build-arg ALASQL_VERSION=4.3.2 -t my-image:4.3.2 . && docker run my-image:4.3.2

Output: The current value of utils.isNode is: true

Thank you both.