test: increase `no-errors` integration timeout to 20s - Githubissues

ezolenko / rollup-plugin-typescript2

Rollup plugin for typescript with compiler errors.

MIT License

819 stars 71 forks source link

test: increase `no-errors` integration timeout to 20s #425

Closed agilgur5 closed 1 year ago

agilgur5 commented 1 year ago

Summary

Increase the timeout on the no-errors integration test suite

Follow-up to #371, where, per the commits, I bumped it several times due to CI timeouts

Details

this was still occasionally failing on CI at 15s, so bump up a bit
- c.f. this run, this run, this run, and many more
- only no-errors was timing out, which makes sense, since it does 3 builds for the first test to test against the cache
- this potentially could be optimized, but it started getting complicated to do so (including some concurrency issues) - so punting on that for now and just increasing the timeout

Note that these have never timed out on me locally, only in CI 🤷