delta/node-new.git/test/report, branch Remove-Python-3-tests

report: modify getReport() to return an Object

2019-07-12T21:48:09+00:00

It's likely that anyone using `process.report.getReport()` will be
processing the return value thereafter (e.g., filtering fields or
redacting secrets). This change eliminates boilerplate by calling
`JSON.parse()` on the return value.

Also modified the `validateContent()` and `validate()` test helpers in
`test/common/report.js` to be somewhat more obvious and helpful. Of
note, a report failing validation will now be easier (though still not
_easy_) to read when prepended to the stack trace.

- Refs: https://github.com/nodejs/diagnostics/issues/315

PR-URL: https://github.com/nodejs/node/pull/28630
Reviewed-By: Anna Henningsen 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Jiawen Geng 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Rich Trott

test: check getReport when error with one line stack

2019-07-06T05:47:05+00:00

PR-URL: https://github.com/nodejs/node/pull/28433
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Ruben Bridgewater

test: check writeReport when error with one line stack

2019-07-06T05:47:03+00:00

PR-URL: https://github.com/nodejs/node/pull/28433
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Ruben Bridgewater

test: better output for test-report-uv-handles.js

2019-05-01T20:36:02+00:00

PR-URL: https://github.com/nodejs/node/pull/27479
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig

report: print common items first for readability

2019-04-27T10:04:36+00:00

PR-URL: https://github.com/nodejs/node/pull/27367
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Yongsheng Zhang

lib: replace --diagnostic-report-* with --report-*

2019-04-22T11:09:52+00:00

In the code base the word `report` is almost only used to refer to
the diagnostic report when it's a noun, and it's programmable
interface `process.report()` it not prefixed, so `report` should be
unambiguous enough to use without `diagnostic`.

PR-URL: https://github.com/nodejs/node/pull/27312
Reviewed-By: Gireesh Punathil 
Reviewed-By: Colin Ihrig

report: improve signal name validation

2019-04-11T00:36:10+00:00

PR-URL: https://github.com/nodejs/node/pull/27137
Reviewed-By: Richard Lau 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Jeremiah Senkpiel

report: add support for UDP connected sockets

2019-04-05T17:24:06+00:00

PR-URL: https://github.com/nodejs/node/pull/27072
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Santiago Gimeno

report: rename triggerReport() to writeReport()

2019-03-10T21:16:13+00:00

writeReport() is more descriptive of what the function does.

PR-URL: https://github.com/nodejs/node/pull/26527
Reviewed-By: Richard Lau 
Reviewed-By: Ruben Bridgewater

test: cover triggerReport() failure case

2019-03-10T20:08:57+00:00

PR-URL: https://github.com/nodejs/node/pull/26524
Reviewed-By: Richard Lau 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Daijiro Wachi 
Reviewed-By: James M Snell