False positive build result in vcbuild.bat

[gengjiawen]

https://github.com/nodejs/node/pull/31562/checks?check_run_id=414343479 is showing as passed but it actually failed:

Run ./vcbuild.bat
Looking for Python
Python found in C:\hostedtoolcache\windows\Python\3.6.8\x64\\python.exe
Looking for NASM
Looking for Visual Studio 2019
calling: "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\\Auxiliary\Build\vcvarsall.bat" amd64
**********************************************************************
** Visual Studio 2019 Developer Command Prompt v16.4.3
** Copyright (c) 2019 Microsoft Corporation
**********************************************************************
[vcvarsall.bat] Environment initialized for: 'x64'
Found MSVS version 16.0
configure  --dest-cpu=x64
Node.js configure: Found Python 3.6.8...
Traceback (most recent call last):
  File "configure", line 26, in <module>
    import configure
  File "D:\a\node\node\configure.py", line 1770, in <module>
    run_gyp(gyp_args)
  File "tools\gyp_node.py", line 54, in run_gyp
    rc = gyp.main(args)
  File "tools\gyp\pylib\gyp\__init__.py", line 547, in main
    return gyp_main(args)
  File "tools\gyp\pylib\gyp\__init__.py", line 523, in gyp_main
    options.duplicate_basename_check)
   File "tools\gyp\pylib\gyp\__init__.py", line 107, in Load
    generator.CalculateVariables(default_variables, params)
  File "tools\gyp\pylib\gyp\generator\msvs.py", line 1927, in CalculateVariables
    generator_flags.get('msvs_version', 'auto'))
  File "tools\gyp\pylib\gyp\MSVSVersion.py", line 446, in SelectVisualStudioVersion
    versions = _DetectVisualStudioVersions(version_map[version], 'e' in version)
KeyError: '2019'
Failed to create vc project files.

Originally posted by @richardlau in #31562 (comment)

cc @nodejs/platform-windows

[richardlau]

from #31562 (comment):

I don't have time to look into it but we should look into why the job passed when it failed to run the configure script. (If someone does have time to look into it maybe see whether running the batch file via PowerShell is losing the exit code from the batch file.)

We can see that the job ran vcbuild.bat in PowerShell:
https://github.com/nodejs/node/pull/31562/checks?check_run_id=414343479#step:5:3

  shell: C:\Program Files\PowerShell\6\pwsh.EXE -command ". '{0}'"