kondratyev-nv / vscode-python-test-adapter

Python Test Adapter for the VS Code Test Explorer
https://marketplace.visualstudio.com/items?itemName=LittleFoxTeam.vscode-python-test-adapter
MIT License
117 stars 27 forks source link

Include process.env.PYTHONPATH when creating PYTHONPATH in pytestTestRunner.ts #237

Closed bergkvist closed 3 years ago

bergkvist commented 3 years ago

Fixes: https://github.com/kondratyev-nv/vscode-python-test-adapter/issues/230

bergkvist commented 3 years ago

The failing tests seem to have failed with some kind of timeout:

Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (d:\a\1\s\out\test\tests\pytestScript.test.js)
Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (d:\a\1\s\out\test\tests\pytestScript.test.js)
    at listOnTimeout (internal/timers.js:549:17)
    at processTimers (internal/timers.js:492:7)

image

bergkvist commented 3 years ago

I don't seem to be able to run the test suite locally. Is this an error message you've seen before?

$ npm test

> vscode-python-test-adapter@0.6.7 pretest /home/tobias/repos/vscode-python-test-adapter
> npm run build

> vscode-python-test-adapter@0.6.7 build /home/tobias/repos/vscode-python-test-adapter
> tsc

> vscode-python-test-adapter@0.6.7 postbuild /home/tobias/repos/vscode-python-test-adapter
> copyfiles "resources/python/**/*.py" out

> vscode-python-test-adapter@0.6.7 test /home/tobias/repos/vscode-python-test-adapter
> cross-env SOME_PROCESS_VARIABLE=HelloFromProcessEnv cross-env CODE_TESTS_WORKSPACE=./test/test_samples/samples-workspace.code-workspace node ./out/test/vscode-runner.js

Python executable is python
Using python /usr/bin/python 3.8.5 (default, Jul 28 2020, 12:59:40) 
[GCC 9.3.0]

Found .vscode-test/vscode-1.55.2. Skipping download.
The futex facility returned an unexpected error code.

Exit code:   null
Done

SIGABRT
Failed to run tests
npm ERR! Test failed.  See above for more details.
kondratyev-nv commented 3 years ago

The futex facility returned an unexpected error code.

Weird... Never saw this issue before. Well, I'll merge it and try to add some tests by myself. Thanks for working on this!

bergkvist commented 3 years ago

Okay, thanks @kondratyev-nv :)