SFTtech / openage

Free (as in freedom) open source clone of the Age of Empires II engine 🚀

http://openage.dev

Other

12.6k stars 1.11k forks source link

Create test cases for existing subsystems #189

Open TheJJ opened 9 years ago

TheJJ commented 9 years ago

Because reasons we should add more tests to the code.

Substems with test coverage:

[X] Pathfinding
[x] Jobmanager
[ ] Terrain blending
[x] Coordinate system
[ ] Asset manager
[ ] Logger

For the missing ones, please create some creative tests. An example how that could look like can be seen in libopenage/datastructure/.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

sreejithr commented 9 years ago

Hey, I can write unit tests for you. I'm still reading up the codebase. But I'll start right away. Anything in particular you guys want from me?

TheJJ commented 9 years ago

@sreejithr: Whatever test you want to implement, go for it :smile:. If you need any help on the codebase the best would be joining our IRC and asking there.

TheJJ commented 9 years ago

197 will benefit greatly from this.

ghost commented 9 years ago

Created tests for pathfinding logic in pull request #241

kukinsula commented 9 years ago

pull request for testing the file sink logging: https://github.com/SFTtech/openage/pull/309

sreejithr commented 9 years ago

Made existing testcases for JobManager test with atleast 10 jobs in parallel (https://github.com/SFTtech/openage/pull/310).

simonsan commented 5 years ago

This is an evergreen to raise testing coverage of our code. Also it is a good first issue and we can just do it. I will pin this issue to bring more attention to it for new people looking into the issues that want to do stuff.

simonsan commented 4 years ago

An example that has been merged already can be seen here: https://github.com/SFTtech/openage/pull/241/files