Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

node tests fail if run inside nox #139

Closed
joemarshall opened this issue Jul 11, 2024 · 1 comment
Closed

node tests fail if run inside nox #139

joemarshall opened this issue Jul 11, 2024 · 1 comment

Comments

@joemarshall
Copy link
Collaborator

If I run tests for urllib3 against node.js they fail. This is because it uses nox, which in turn runs pytest as a subprocess. Somewhere along the way nox manages to do something to the tty so that NodeRunner.run_js only sends the first 4096 characters of the javascript along.

I can see that we call stty -icanon to try and stop this happening, but for some reason in this case it fails.

Possibly it would make sense to split the strings into e.g. 128 character lines, and then concatenate them to the code string without adding a \n in the javascript test runner unless it receives an entirely blank line which can be a \n. That way it should work whatever the tty settings.

@ryanking13
Copy link
Member

I guess we can close this? @joemarshall

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants