openage icon indicating copy to clipboard operation
openage copied to clipboard
verified publisher icon SFTtech

Create test cases for existing subsystems

Open TheJJ opened this issue 11 years ago • 8 comments

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.

TheJJ avatar Jan 08 '15 14:01 TheJJ

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?

sreejithr avatar Jan 19 '15 18:01 sreejithr

@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 avatar Jan 19 '15 22:01 TheJJ

#197 will benefit greatly from this.

TheJJ avatar Feb 07 '15 14:02 TheJJ

Created tests for pathfinding logic in pull request #241

ghost avatar Mar 03 '15 00:03 ghost

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

kukinsula avatar May 28 '15 12:05 kukinsula

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

sreejithr avatar May 30 '15 11:05 sreejithr

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 avatar Jul 01 '19 08:07 simonsan

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

simonsan avatar Feb 15 '20 22:02 simonsan