Skip to content

Add a timeout to ProcessStartedHandler #4392

@aishwaryabh

Description

@aishwaryabh

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)

Metadata

Metadata

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions