node
node copied to clipboard
test: remove flaky designation
Searching the automatically-generated test failure reports in the issues at https://github.com/nodejs/reliability/issues, I don't see test-worker-message-port-message-before-close
failing any time recently.
Isn't it because of https://github.com/nodejs/build/pull/3598?
There should be failures in https://github.com/nodejs/reliability/issues before https://github.com/nodejs/build/pull/3598 landed, no?
CI: https://ci.nodejs.org/job/node-test-pull-request/57093/
CI: https://ci.nodejs.org/job/node-test-pull-request/57106/
CI: https://ci.nodejs.org/job/node-test-pull-request/57132/
CI: https://ci.nodejs.org/job/node-test-pull-request/57302/
CI: https://ci.nodejs.org/job/node-test-pull-request/57387/
CI: https://ci.nodejs.org/job/node-test-pull-request/57441/
I don't think the reliability report considers tests that are marked as flaky: https://github.com/nodejs/node-core-utils/blob/0d49edaf6736b393b0597ee67d70381cd5841b40/lib/ci/ci_failure_parser.js#L157-L170 (perhaps we should, for the walk command used by the reliability report), but as far as I've seen it's not been failing recently.
CI: https://ci.nodejs.org/job/node-test-pull-request/57470/
CI: https://ci.nodejs.org/job/node-test-pull-request/57472/
CI: https://ci.nodejs.org/job/node-test-pull-request/57475/
CI: https://ci.nodejs.org/job/node-test-pull-request/57493/
CI: https://ci.nodejs.org/job/node-test-pull-request/57513/
CI: https://ci.nodejs.org/job/node-test-pull-request/57515/
CI: https://ci.nodejs.org/job/node-test-pull-request/57539/
CI: https://ci.nodejs.org/job/node-test-pull-request/57544/
CI: https://ci.nodejs.org/job/node-test-pull-request/57545/
Landed in 5c97f411d813b5d9783ced28fe3f968f568e1ffa