Repository navigation
Tests that rely upon common.spawnCat() faill in Windows with Unicode shell codepage #11470
Description
- Version: possibly all
- Platform: Windows 7 x64
- Subsystem: test, child_process
For context see #11469
test/parallel/test-child-process-stdio-inherit.js calls common.spawnCat() that calls more for Windows. more fails with Unicode shell codepage.
To check test/parallel/test-child-process-stdio-inherit.js pass:
chcp 65001 && node test\parallel\test-child-process-stdio-inherit.js
Active code page: 65001
Not enough memory.
... AssertionError: '' === 'asdfasdf' ...
chcp 1252 && node test\parallel\test-child-process-stdio-inherit.js
Active code page: 1252
[pass]
It seems currently this is the only test that uses common.spawnCat().
Activity
joaocgreis commentedon Feb 21, 2017
Can't this be addressed as a duplicate of #11469 ? If I understand correctly, this is for
spawnCatwhile #11469 is formoredirectly, but given thatspawnCatusesmore, the underlying issue is the same, right?vsemozhetbyt commentedon Feb 21, 2017
@joaocgreis Yeah, I just was not sure if I should combine the issue with the lib and the issue with individual tests. If anybody finds a universal solution, please, close the both issues. Or feel free to close any of them as a duplicate in advance.
joaocgreis commentedon Feb 22, 2017
Closing as a duplicate of #11469 .
Thanks @vsemozhetbyt ! If this was an issue in
/libthen it would be an issue for users and would have to be addressed differently, but/test/common.jsis only used by the tests, so I think this is essentially the same problem.