Node: test: fix/rename test/debugger/helper-debugger-*

[refack]

• Version: master
• Subsystem: test

tests for line debugger should use inspect protocol and CLI args

Tracking:

• [ ] test/debugger/test-debugger-repl-break-in-module.js
• [ ] test/debugger/test-debugger-repl-restart.js
• [ ] test/debugger/test-debugger-repl-term.js
• [ ] test/debugger/test-debugger-repl-utf8.js
• [ ] test/debugger/helper-debugger-repl.js
• [ ] test/debugger/test-debugger-repl.js

[Trott]

These tests aren't run in CI or via make test/vcbuild test. They should probably be moved one-by-one to parallel or sequential if they work there, and ones that are truly debugger-specific should be removed. They have a very checkered history in terms of reliability.

[refack]

To follow up on @Trott suggestion I've made a tracking list. Marking as good-first-contribution but I assume contributors will need mentoring. I'll be happy to help.

[rhoboat]

dibs

[refack]

@archanid as I said, hopefully it's well defined, but it might require some trial and error. If you need any help feel free to ping me.

[maclover7]

@refack @Trott should this be closed since #15139 was merged / test/debugger has been deleted?

[refack]

@archanid I'm closing this. If by any chance you found some good nuggets in that directory, feel free to PR them back.