This PR adds a couple of integration tests to the worker.
These tests ensure that:
a) the default timeout on the worker, set the env var, is respected
b) a timeout can be set on a run and that will overidde the default
One thing that is NOT covered is the env var itself. But I have manually checked that the env var is being passed through correctly, and server startup logs should verify the actual value being used (note that this is an ENGINE option, not a WORKER option)
This PR adds a couple of integration tests to the worker.
These tests ensure that: a) the default timeout on the worker, set the env var, is respected b) a timeout can be set on a run and that will overidde the default
One thing that is NOT covered is the env var itself. But I have manually checked that the env var is being passed through correctly, and server startup logs should verify the actual value being used (note that this is an ENGINE option, not a WORKER option)
No release necessary, we can just merge this.
Closes #686