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
After merging in this PR, when the tests are run locally today for core tools for func start, if the process starts up and is not interrupted within the test by the processStartedHandler (example: we expect the test to fail and throw an error before the host starts up, but it doesn't so the test is stuck since the host starts up successfully and is waiting for the user to manually kill the process), the test stalls, which is a flaw in the design.
To workaround this, a timeout would need to be added to the processTask here. If the timeout is reached before processStarted is finished, we have to manually kill the process by doing _process.Kill(true)
The text was updated successfully, but these errors were encountered:
After merging in this PR, when the tests are run locally today for core tools for
func start
, if the process starts up and is not interrupted within the test by the processStartedHandler (example: we expect the test to fail and throw an error before the host starts up, but it doesn't so the test is stuck since the host starts up successfully and is waiting for the user to manually kill the process), the test stalls, which is a flaw in the design.To workaround this, a timeout would need to be added to the processTask here. If the timeout is reached before
processStarted
is finished, we have to manually kill the process by doing_process.Kill(true)
The text was updated successfully, but these errors were encountered: