Skip to content

test: add test-debugger-breakpoint-exists

Rodrigo Muino Tomonari requested to merge github/fork/Trott/add-debug-stuff into master
  • The data parameter of unpackError() is typically undefined. Remove it.
  • In Node.js 15, calling setBreakpoint(1) and restart twice in a row caused the debugger to exit. In Node.js 16, it no longer exits but throws an error that is expected, or at least reasonable, or at least better than exiting.
  • The error message previously had undefined appended to it. It no longer does.
  • This adds test coverage to unpackError() in lib/internal/debugger/inspect_client.js. That function previously was untested.

Refs: nodejs/node#41789

Merge request reports