Fix child_process test in node <16

[uurien]

What does this PR do?

Fix the failing CI with child_process plugins in node <=16 release proposals.

PR with the same changes with CI running in node <=16: #4058

And the CI result in green for child_process: https://github.com/DataDog/dd-trace-js/actions/runs/7900095414/job/21560938934?pr=4058

Motivation

When child_process.execFileSync() is executed in node <=16, with { shell: true } and the executed application returns an error, it doesn't throw an execption and the result is an expected Buffer, we can't detect the error there, so we can't test it.

[github-actions]

Overall package size

Self size: 6.04 MB
Deduped: 61.64 MB
No deduping: 62.39 MB

Dependency sizes

name	version	self size	total size
@datadog/native-iast-taint-tracking	1.6.4	16.43 MB	16.44 MB
@datadog/native-appsec	7.0.0	14.51 MB	14.52 MB
@datadog/pprof	5.0.0	9.59 MB	10.44 MB
protobufjs	7.2.5	2.77 MB	6.56 MB
@datadog/native-iast-rewriter	2.2.3	2.19 MB	2.28 MB
@opentelemetry/core	1.14.0	872.87 kB	1.47 MB
@datadog/native-metrics	2.0.0	898.77 kB	1.3 MB
@opentelemetry/api	1.4.1	780.32 kB	780.32 kB
import-in-the-middle	1.7.3	67.62 kB	731.01 kB
pprof-format	2.0.7	588.12 kB	588.12 kB
msgpack-lite	0.1.26	201.16 kB	281.59 kB
opentracing	0.14.7	194.81 kB	194.81 kB
semver	7.5.4	93.4 kB	123.8 kB
@datadog/sketches-js	2.1.0	109.9 kB	109.9 kB
lodash.sortby	4.7.0	75.76 kB	75.76 kB
lru-cache	7.14.0	74.95 kB	74.95 kB
ipaddr.js	2.1.0	60.23 kB	60.23 kB
ignore	5.2.4	51.22 kB	51.22 kB
int64-buffer	0.1.10	49.18 kB	49.18 kB
shell-quote	1.8.1	44.96 kB	44.96 kB
istanbul-lib-coverage	3.2.0	29.34 kB	29.34 kB
tlhunter-sorted-set	0.1.0	24.94 kB	24.94 kB
limiter	1.1.5	23.17 kB	23.17 kB
dc-polyfill	0.1.4	23.1 kB	23.1 kB
retry	0.13.1	18.85 kB	18.85 kB
node-abort-controller	3.1.1	16.89 kB	16.89 kB
jest-docblock	29.7.0	8.99 kB	12.76 kB
crypto-randomuuid	1.0.0	11.18 kB	11.18 kB
path-to-regexp	0.1.7	6.78 kB	6.78 kB
koalas	1.0.2	6.47 kB	6.47 kB
methods	1.1.2	5.29 kB	5.29 kB
module-details-from-path	1.0.3	4.47 kB	4.47 kB

_{🤖 This report was automatically generated by heaviest-objects-in-the-universe}

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1bdacab) 85.33% compared to head (0f317ae) 85.33%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4059   +/-   ##
=======================================
  Coverage   85.33%   85.33%           
=======================================
  Files         243      243           
  Lines       10569    10569           
  Branches       33       33           
=======================================
  Hits         9019     9019           
  Misses       1550     1550           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-02-14 13:04:29

Comparing candidate commit 0f317ae in PR branch ugaitz/fix-child_process-in-node-v16 with baseline commit 1bdacab in branch master.

Found 2 performance improvements and 0 performance regressions! Performance is the same for 259 metrics, 5 unstable metrics.

scenario:plugin-graphql-with-depth-off-18

• 🟩 max_rss_usage [-134.235MB; -119.113MB] or [-13.920%; -12.351%]

scenario:plugin-graphql-with-depth-on-max-18

• 🟩 max_rss_usage [-160.656MB; -95.860MB] or [-16.721%; -9.977%]

[juan-fernandez]

LGTM. Integration tests are failing though. Is it flakiness?

[uurien]

@juan-fernandez

LGTM. Integration tests are failing though. Is it flakiness?

It is failing also in master, I'm merging this.