test: log MSBuild output on Windows build failures

[legendecas]

Checklist

• npm install && npm run lint && npm test passes
• tests are included
• documentation is changed or added
• commit message follows commit guidelines

Description of change

Refs: #3327

[legendecas]

The error seems to be:

  LINK : fatal error LNK1117: syntax error in option 'opt:lldltojobs=2' [D:\a\node-gyp\node-gyp\test\node_modules\hello_world\build\hello.vcxproj]

    20 Warning(s)
    1 Error(s)

[StefanStojanovic]

From what I understand, the idea behind this draft PR is that you wanted to see what the problem was with Node.js v26 tests on Windows. The question I have is, do we want to keep this change to be able to get more information in the future or would that not be useful to have?

[legendecas]

From what I understand, the idea behind this draft PR is that you wanted to see what the problem was with Node.js v26 tests on Windows. The question I have is, do we want to keep this change to be able to get more information in the future or would that not be useful to have?

Yes, I think this is generally helpful and worth to keep.

[cclauss]

Please fix order of return values.

[legendecas]

CI failures should be fixed by #3345.

[cclauss]

All checks have passed.

[legendecas]

Thanks!