silverstripe-archive / silverstripe-forum

Forum module for SilverStripe CMS
http://silverstripe.org/forum-module
Other
34 stars 60 forks source link

Addition of Test Coverage #190

Closed gordonbanderson closed 7 years ago

gordonbanderson commented 8 years ago

Added test coverage and badges for the README. I've also added checks for SilverStripe 3.2 and 3.3 which both fail under php 5.6. However I am not sure why php 5.6 is allowed to fail, but I have left it as is for now. As such the build still passes :)

codecov-io commented 8 years ago

Current coverage is 53.74%

Branch #190 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found. Review entire Coverage Diff as of 66562b6

Powered by Codecov. Updated on successful CI builds.

camfindlay commented 8 years ago

Cheers for this! Just awaiting a fix up of tests for 3.3 core over in #186 once that is merge I think it will sort the test related issues out.

gordonbanderson commented 8 years ago

It should be noted that the figure of 53.7% coverage is misleading in that the code/search directory classes are not tested, and don't count towards the final percentage.