search

beyond-all-reason / teiserver

Middleware server for online gaming
https://www.beyondallreason.info/
MIT License
50 stars 47 forks source link

Some smol test fixing #243

Closed geekingfrog closed 5 months ago

geekingfrog commented 5 months ago

I'm taking baby steps to understand teiserver, and I figured fixing some simple tests first would be a good first step. It seems because there's no CI lots of tests have bitrot and many seems to be failing for some trivial reasons, I'll get to others soon™

On latest master: 413 tests, 151 failures

On this branch: 414 tests, 148 failures

Beherith commented 5 months ago

@L-e-x-o-n has been doing a lot of work on making the tests pass :)

L-e-x-o-n commented 5 months ago

Yes, working on some new ones in #244

geekingfrog commented 5 months ago

I think we should slowly merge these fixes in instead of trying to have one or two big PR to fix everything. That could also motivate others to join in the effort?

L-e-x-o-n commented 5 months ago

I think that's a good idea, avoid duplication of work and possible merge conflicts.