go-plus icon indicating copy to clipboard operation
go-plus copied to clipboard

Configurable Guru Timeout

Open robbawebba opened this issue 6 years ago • 3 comments

This commit adds a user-configurable timeout value to all guru commands.

closes #951

Two concerns I would like to address about the proposed changes:

  • I am encountering an issue with guru when running tests locally (both on my branch and on the HEAD of master): guru: no initial packages were loaded. I'm unsure if this is due to my local dev environment, so I wanted to see if the tests pass in the CI environment.
  • It is proving to be very difficult to test the highlight provider guru timeout. this guru operation seems to work pretty quickly in the test environment. When I set a timeout of 1ms in the package settings during normal usage, the command does not seem to finish within 1ms and no identifiers in the editor are highlighted. when I set a timeout of 1ms in a spec, the highlight provider's highlight function seems to finish before the timeout, and valid highlight ranges are returned by the function. I'm not sure if it's going to be possible to test the timeout of the highlight provider.

robbawebba avatar Oct 07 '19 05:10 robbawebba

Looks like a couple tests are broken, but otherwise LGTM.

zmb3 avatar Oct 07 '19 14:10 zmb3

Hi @zmb3, thanks for taking a look! I'm running into issued with running the specs locally, which makes the issues difficult to debug. I'm running several issues with guru locally that do not seem to be a problem in the CI environment. For example, most guru commands that are run through the specs fail with this error:

guru: no initial packages were loaded

Have you seen this before while running tests locally?

robbawebba avatar Oct 07 '19 20:10 robbawebba

I haven't seen that locally, though I admit it's been a while since I've tried. That sounds to me like a gopath or working directory problem.

zmb3 avatar Oct 15 '19 00:10 zmb3