[v9.x backport] build: add node_lib_target_name to cctest deps

[danbev]

Currently the cctest target depend on the node_core_target_name
target. But it is the node_lib_target_name target that compiles the
sources now which means that if a source file in src is updated the
cctest executable will not be re-linked against it, but will remain
unchanged. The code will still be compiled, just not linked which
means that if you are debugging you'll not see the changes and also a
warning will be displayed about this issue.

This commit changes the cctest target to depend on node_lib_target_name.

PR-URL: #18576
Reviewed-By: Matheus Marchini [email protected]
Reviewed-By: Yihong Wang [email protected]
Reviewed-By: Gibson Fahnestock [email protected]
Reviewed-By: Ben Noordhuis [email protected]

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• commit message follows commit guidelines

Affected core subsystem(s)

build

[danbev]

CI: https://ci.nodejs.org/job/node-test-pull-request/13345/

[mmarchini]

@danbev this will break once #18550 lands. Should I add a commit to #18550 with the same fix you did on #18576?

[danbev]

Should I add a commit to #18550 with the same fix you did on #18576?

That sounds great! Let me know if we can close this in that case. Thanks

[mmarchini]

Commit added to #18550. @danbev I think we can close this now (no need to resolve the conflict since I'll fix it on #18550)

[danbev]

@mmarchini Great, thanks for that. Will close now.