Skip to content

Tests that call more fail in Windows with Unicode shell codepage #11469

Closed
Closed
Tests that call more fail in Windows with Unicode shell codepage#11469
Labels
testIssues and PRs related to Node.js core tests and test infrastructure.windowsIssues and PRs related to the Windows platform.

Description

@vsemozhetbyt
vsemozhetbyt
opened on Feb 20, 2017
  • Version: possibly all
  • Platform: Windows 7 x64
  • Subsystem: test, child_process

test\parallel\test-child-process-stdin.js uses more command and does not expect any error messages in the stderr (see common.mustNotCall()).

However, some core Windows shell commands work wrong with Unicode shell codepage. See this answer.

To check more command:

chcp 65001 && more
Active code page: 65001
Not enough memory.

chcp 1252 && more
Active code page: 1252
[reading from stdio]

To check test\parallel\test-child-process-stdin.js pass:

chcp 65001 && node test\parallel\test-child-process-stdin.js
Active code page: 65001
... AssertionError: function should not have been called ...

chcp 1252 && node test\parallel\test-child-process-stdin.js
Active code page: 1252
stdout: hello world

Activity

  1. addaleax
    added
    testIssues and PRs related to Node.js core tests and test infrastructure.
    windowsIssues and PRs related to the Windows platform.
    on Feb 20, 2017
  2. vsemozhetbyt
    changed the title [-]test\parallel\test-child-process-stdin.js fails in Windows with Unicode shell codepage[/-] [+]Tests that call `more` fail in Windows with Unicode shell codepage[/+] on Feb 20, 2017

  3. vsemozhetbyt commented on Feb 21, 2017

    @vsemozhetbyt
    vsemozhetbyt
    on Feb 21, 2017
    ContributorAuthor

    It seems the same with these two tests:

    test\parallel\test-child-process-spawn-shell.js calls more here.

    test\parallel\test-child-process-spawnsync-shell.js calls more here.

    The same checks:

    chcp 65001 && node test\parallel\test-child-process-spawnsync-shell.js
... AssertionError: '' === 'bar' ...

chcp 1252 && node test\parallel\test-child-process-spawnsync-shell.js
[pass]

chcp 65001 && node test\parallel\test-child-process-spawn-shell.js
... AssertionError: '' === 'bar' ...

chcp 1252 && node test\parallel\test-child-process-spawn-shell.js
[pass]

  4. seishun commented on Feb 21, 2017

    @seishun
    seishun
    on Feb 21, 2017
    Contributor

    So I guess it should be documented that tests should be run in a non-Unicode codepage.

  5. vsemozhetbyt commented on Feb 21, 2017

    @vsemozhetbyt
    vsemozhetbyt
    on Feb 21, 2017
    ContributorAuthor

    Or maybe the test runner (or a concerned test / common lib) should set an appropriate codepage before a run.

  6. joaocgreis commented on Feb 21, 2017

    @joaocgreis
    joaocgreis
    on Feb 21, 2017
    Member

    I can reproduce this on Windows 7 and 2008R2 but cannot on Windows Server 2012R2. So, this looks like an issue with more (that ships with Windows: C:\Windows\System32\more.com) that has been fixed on Windows 8/2012.

    I suspect more is used just because it was never changed to cat, we already require cat (installed by git) to run the tests. I'd be happy to discuss a PR to s/more/cat/, if it works.

  7. TimothyGu commented on Feb 21, 2017

    @TimothyGu
    TimothyGu
    on Feb 21, 2017
    Member

    I can't reproduce this on Windows 8.1 (6.3.9600). chcp 65001 && more works just fine. Changing to cat SGTM.

  8. vsemozhetbyt commented on Feb 22, 2017

    @vsemozhetbyt
    vsemozhetbyt
    on Feb 22, 2017
    ContributorAuthor

    See also: #11470

  9. TimothyGu commented on Mar 20, 2017

    @TimothyGu
    TimothyGu
    on Mar 20, 2017
    Member

    @vsemozhetbyt do you want to submit a PR for this?

  10. vsemozhetbyt commented on Mar 20, 2017

    @vsemozhetbyt
    vsemozhetbyt
    on Mar 20, 2017
    ContributorAuthor

    @TimothyGu If @joaocgreis has no plans to do this, I can try. However, I am not good at *nix shell tools so I can overlook something.

  11. joaocgreis commented on Mar 20, 2017

    @joaocgreis
    joaocgreis
    on Mar 20, 2017
    Member

    I've added this to my queue, but I'm not likely to get there soon. Feel free to go ahead, I can help review.

  12. vsemozhetbyt commented on Mar 21, 2017

    @vsemozhetbyt
    vsemozhetbyt
    on Mar 21, 2017
    ContributorAuthor

    @joaocgreis The PR, PTAL.

  13. vsemozhetbyt
    closed this as completedin 2dff3a2on Mar 23, 2017
  14. MylesBorins
    added a commit that references this issue on Mar 28, 2017

    test: do not use `more` command on Windows

    bb2de4a
  15. sjarmak
    added a commit that references this issue on Jul 27, 2026

    test: do not use `more` command on Windows

    2dff3a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    testIssues and PRs related to Node.js core tests and test infrastructure.windowsIssues and PRs related to the Windows platform.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions