-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
pytest discovery fails over ssh if there are too many tests #22421
Comments
Are you on the new testing rewrite? You can check for it by looking at |
I do see in my Python output window, near the top:
I never turned it on manually, and it says I see it in both vscode versions: 1.83 and 1.84 |
could you include your full logs from the |
The problem seems to be the quantity of the unit tests:
If I ignore the folder with 80k unit tests, it works The size of the log files and the log rotation seems to desync the python output window as well. It frequently doesn't switch to the latest log file |
oh. I waited 10 minutes, and the tests did show up in the tests sidebar. I guess it does work, it's just slow. Also, I get a lot of "window not responding" popups whenever I load the sidebar. maybe unit test refresh should run in a non-blocking way. doubt that's possible without a rework |
here's a file that reproduces all the problems:
|
Hello! I have just returned from vacation and will take a look at this shortly, thank you for your patience! |
@roblourens any thoughts here? Seems like a slow running process run by an extension is causing a repeat warning message "window not responding" as it is working. Is this something I can change on my end to demonstrate a process is under way and not get this pop up? |
Either the test sidebar UI is loading tests in an inefficient way, or the extension is sending them in an inefficient way cc @connor4312 can you load 80k tests? |
In my testing, loading the 75k tests from the lsprotocol repo was reasonably speedy, with less than a second on the renderer, though the extension host took longer (#22414). I observed the same with the same test case. A renderer profile would be useful here https://github.com/microsoft/vscode/wiki/Performance-Issues#profiling-the-renderer-process |
Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on, then we will be happy to re-open this issue to pick up where we left off. Happy Coding! |
Type: Bug
Behaviour
Expected vs. Actual
Pytest unit tests are not showing up in the sidebar, when using a remote ssh. In the python output window, I see the pytest --collect-only command ran, and produced the expected output. However, the unit test sidebar still says "no tests have been found in this workspace"
Steps to reproduce:
Windows, VSCode 1.84 (October 2023)
Ubuntu vm
Python extension v2023.20.0
SSH extension v0.106.5
If I downgrade VSCode, the ssh extension, and the python extension to the September 2023 version, unit tests work again
Diagnostic data
python.languageServer
setting: DefaultOutput for
Python
in theOutput
panel (View
→Output
, change the drop-down the upper-right of theOutput
panel toPython
)User Settings
Extension version: 2023.20.0
VS Code version: Code 1.83.1 (f1b07bd25dfad64b0167beb15359ae573aecd2cc, 2023-10-10T23:48:05.904Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Remote OS version: Linux x64 5.15.0-88-generic
System Info
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
A/B Experiments
The text was updated successfully, but these errors were encountered: