-
Notifications
You must be signed in to change notification settings - Fork 30
Refactoring + Unit tests #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Also start refactoring everything, because some of this stuff is a mess. Also also: fix some tile data based on errors caught by the unit tests. So that works, hurray!
1481eb0 to
4ab0b27
Compare
End is in sight here...
|
Currently betatesting, courtesy of the fine TI community on reddit.
|
We're finally professionalizing up in here, because my I-threw-this-over-the-weekend style of architecture is causing problems. 🫠
This started as a pretty innocent attempt to introduce Unit testing into the project, but soon grew into a full refactor of pretty much the entire back-end.
I realised that in order to do Unit testing, we need units, and the classes and implementations now were way to huge to be tested effectively.
This refactor aims to fix the following things: