Integration tests timeout: workspace-namespace registerTextDocumentContentProvider, change event

[joaomoreno]

Repros not so often, but just happened:

2018-04-19T08:57:46.6886760Z �[92m �[0m�[32m 82 passing�[0m�[90m (1m)�[0m
2018-04-19T08:57:46.6912300Z �[31m  1 failing�[0m
2018-04-19T08:57:46.6925370Z 
2018-04-19T08:57:46.6950290Z �[0m  1) workspace-namespace registerTextDocumentContentProvider, change event:
2018-04-19T08:57:46.6980410Z �[0m�[31m     Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.�[0m�[90m

https://monacotools.visualstudio.com/Monaco/_build/index?buildId=38404&_a=summary

[bpasero]

I don't think I added that test to be honest. @jrieken ?

[joaomoreno]

Actually just failed again:

https://monacotools.visualstudio.com/Monaco/_build/index?buildId=38405&_a=summary

Maybe there was a new change pushed today/yesterday?

It seems to only fail on macOS.

[jrieken]

I didn't touch anything...

[joaomoreno]

One more failure: https://monacotools.visualstudio.com/Monaco/_build/index?buildId=38456&_a=summary

Though it is still random, it seems it happens much more often now.

[jrieken]

Pushed a change to de-register the listener later. This should/could make this more stable, tho I have also not seen this failing in a while.

[jrieken]

Optimistically closing

[joaomoreno]

Sorry dude :( https://vscode.visualstudio.com/VSCode/_build?buildId=1475

Big coincidence that it just hit today.

[jrieken]

maybe too optimistically... tho I have no clue what's going on...

[jrieken]

No, I really have seen this failing in a long while. Staying optimistically.