Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Confusing error message on save. #1075

Open
idthings opened this issue Dec 9, 2020 · 0 comments
Open

Confusing error message on save. #1075

idthings opened this issue Dec 9, 2020 · 0 comments

Comments

@idthings
Copy link

idthings commented Dec 9, 2020

I am running 'test on save'. My tests run, the output appears in the Output pane. But the Output pane then refreshes and I get:

test timed out after 10000 ms

The timing felt a bit 'async', but I had trouble figuring out which test was running asynchronously. Increasing the tests timeout didn't make a difference.

Then I increased the compile timeout, from 10000 to 90000. This fixed the problem, and the Output pane finished with 'Done'.

I had a look at the build code section, and see my problem. The logic here formats the error string. If hasTests then the build tool name is set to 'test'.

// check for any non-zero exit codes and error if found

I think it would be much better if this was changed to something like:

compile timed out after 10000 ms

I don't have a js/Atom dev env, otherwise I submit a PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant