-
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
tests are not configured after reloading the window #22093
Comments
output:
|
|
question for @meganrogge or @bhavyaus, how long did you both wait to see if the tests would load? Usually what happens with mine is that I click on the test explorer icon then have to wait a few seconds for the screen that says "configure tests" to disappear and be replaced by the tests discovered. This load time is a problem I have noted and I am looking to make a change which I am discussing in this issue: #22007. The issue also has videos demonstrating what I mean by it resolving after a little lag time. If one of you could test to see if this is the case for you and there is just a delay that would be super helpful. Additionally would love your feedback on what UI change should be made to signal that loading is occurring. I was thinking a load icon somewhere (not sure where) but also maybe making it so the "configure tests" button doesn't appear would also help. |
I had waited more than a minute I would say. It's still not discovering tests. here's the full output:
|
I see that "The python test process was terminated before it could exit on its own, the process errored with: Code: null, Signal: SIGSEGV". It seems the issue is it doesn't show an error node- will investigate this. Thanks! |
reviewed this code and still not seeing how it could not produce an error node to display- could you send over the code you were testing it on? I will also add another logging item and see if that gives me more insight in insiders tomorrow. |
we discussed and this happened because there was a segfault in one of my tests. commenting that out fixed it |
I wonder if we could make this error more obvious |
this was a discussion more generally since I am not sure the way it displays it as a node in the test explorer is best- will find that issue and tag it here as we do discuss more. |
feel free to close this if you are tracking it somewhere |
closing in favor of the mentioned issue |
Testing #22081
I was running tests, reloaded the window, tests are no longer discovered.
The text was updated successfully, but these errors were encountered: