integration: add h.complete(false) to integration test failures, bump timeouts to 2s

ponylang / pony-stable

:horse: A simple dependency manager for the Pony language.

BSD 2-Clause "Simplified" License

134 stars 18 forks source link

integration: add h.complete(false) to integration test failures, bump timeouts to 2s #77

Closed srenatus closed 6 years ago

srenatus commented 6 years ago

The reason this timed out could be that it didn't call h.complete(..) when failing. I'm not certain yet, but lets see of this shines some light on it...

Update: didn't seem so, and we should still have seen output. So, assuming that this is a genuine timeout, this bumps the test time outs for all integration test cases to 2s.

srenatus commented 6 years ago

ℹ️ https://circleci.com/gh/ponylang/pony-stable/214 just failed, too, but it's a different env integration test. So this isn't about the nested dir case in any way, but a general issue...

srenatus commented 6 years ago

Let's see if this makes the issue go away or we'll have to dig deeper. Merging...

mfelsche commented 6 years ago

IMHO the most likely cause is simply timeouts, thus increasing the timeouts should fix this for now.