You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
QL tests in the Test Explorer occasionally go into the "retired" state, where the test result icon is faded. There seems to be little rhyme or reason to when this happens.
Describe the solution you'd like
We should properly implement the retire event on the Test Adapter. It was added to Test Explorer after our original implementation, so it seems to use the default behavior of retiring all tests after any reload of the test tree, which we probably do whenever any file in a test directory changes.
Additional context
See here for some basic information about how the retire event is supposed to work.
The text was updated successfully, but these errors were encountered:
Going to jot down some notes of a slack discussion we had about this:
We will need to delegate to the cli to determine what the dependencies of each test are. We will need to combine that with a filesystem watcher to retire/re-run any test whose dependencies change. It's unclear where these dependency files should live:
a. alongside the tests. One file for each tree. In this case, we need to be careful that we don't accidentally commit these files to VCS.
b. in workspace storage. This will keep these new files out of the workspace tree and will be less confusing for users. But, it will be more complex to maintain consistency between them and the tests.
@dbartol also suggested we could store the expand/collapsed state of the test tree across restarts. We could store that inside of the test hierarchy, or in workspace storage. This is a separate, but related feature.
Is your feature request related to a problem? Please describe.
QL tests in the Test Explorer occasionally go into the "retired" state, where the test result icon is faded. There seems to be little rhyme or reason to when this happens.
Describe the solution you'd like
We should properly implement the
retire
event on the Test Adapter. It was added to Test Explorer after our original implementation, so it seems to use the default behavior of retiring all tests after any reload of the test tree, which we probably do whenever any file in a test directory changes.Additional context
See here for some basic information about how the
retire
event is supposed to work.The text was updated successfully, but these errors were encountered: