Reduce integration test overhead

go-gitea / gitea

Git with a cup of tea! Painless self-hosted all-in-one software development service, including Git hosting, code review, team collaboration, package registry and CI/CD

https://gitea.com

MIT License

45.29k stars 5.51k forks source link

Reduce integration test overhead #32475

Closed bohde closed 1 week ago

bohde commented 2 weeks ago

In profiling integration tests, I found a couple places where per-test overhead could be reduced:

Avoiding disk IO by synchronizing instead of deleting & copying test Git repository data. This saves ~100ms per test on my machine
When flushing queues in PrintCurrentTest, invoke FlushWithContext in a parallel.

wxiaoguang commented 1 week ago

When flushing queues in PrintCurrentTest, invoke FlushWithContext in a parallel.

We could make it even faster (without any waiting time). Could I edit your PR to implement my idea?

bohde commented 1 week ago

We could make it even faster (without any waiting time). Could I edit your PR to implement my idea?

I'd be fine with this!

wxiaoguang commented 1 week ago

Done from my side.

bohde commented 1 week ago

I think these changes look good!

wxiaoguang commented 1 week ago

Need a following up PR: Improve testing and try to fix MySQL hanging #32515