Splits CI Jobs Into Unit And Integration Tests On Linux #169
Conversation
| - swift:5.3 | ||
| - swift:5.4 | ||
| - swiftlang/swift:nightly-5.5 | ||
| - swiftlang/swift:nightly-main |
There was a problem hiding this comment.
This has traditionally caused issues with our CI as we find that bugs in nightly cause our tests to fail and block PRs. I think it would be good to have a separate workflow that runs daily that tests the nightlies
There was a problem hiding this comment.
This is something I was working on last time I had a chance to deal with the CI for more than trivial updates, yeah.
| - name: Check out code | ||
| uses: actions/checkout@v2 | ||
| - name: Run tests with Thread Sanitizer | ||
| run: swift test --enable-test-discovery --sanitize=thread --filter=^PostgresNIOTests |
There was a problem hiding this comment.
I wonder if it's worth trying to run these tests in parallel
There was a problem hiding this comment.
Anything that connects to the database can't run in parallel unless we start having tests generate randomly-named databases (or at least schemas).
There was a problem hiding this comment.
The PostgresNIOTests don't touch a real DB they're all in the integration tests
This was prepared by #157.
CI now also uses more swift compiler versions, namely
5.4,nightly-5.5andnightly-main.It also removes
amazonlinux2and now only tests onfocal.fixes #165