search

waartaa / ircb

A versatile IRC bouncer for humans
MIT License
49 stars 15 forks source link

Proposed Tox and TravisCI for CI/CD #54

Closed slick666 closed 8 years ago

slick666 commented 8 years ago

Here is my implementation of a simple testing scheme that would allow devs to run simple unit tests locally and to have all PRs and code be tested simply and automatically. This implementation isn't supposed to really do much testing at all but provide a framework for the work to be filled in as we go. We can leverage TravisCI to automatically test all PRs as they are proposed and know before merging if something is broken. We can add coverage as we have time or work with the effected code.

rtnpro commented 8 years ago

@slick666 Thanks for the pull request :+1: The changes look fine. I will check the changes at my end before merging it.

Also, will you be interested in setting up the process to test the asyncio based code for ircb?

slick666 commented 8 years ago

@rtnpro I would definitely be interested in getting code coverage for all the code, including the asyncio parts. I'm not sure you cover al the problems with simple unit tests though, I'm thinking some functional tests might be appropriate here. (i.e. this unit runs fine, and this unit runs fine but at the same times something fails.)

Also, I've set this up to run on my own TravisCI and would be willing to help you set up Travis for the project.

slick666 commented 8 years ago

Rebased, should be ready to roll

slick666 commented 8 years ago

@rtnpro @sayanchowdhury can this be merged? Is there anything I can to to help it get merged?

slick666 commented 8 years ago

rebased