search

pearkes / gethub

A utility for cloning and fetching your remote git repositories from GitHub.
MIT License
217 stars 266 forks source link

Add tests for checkRepo #4

Closed vinc closed 11 years ago

vinc commented 11 years ago

Add basic tests for checkRepo().

pearkes commented 11 years ago

Nice! Do you think we can DRY this up a bit for future tests? I feel like we'll be doing it a lot in other places as well.

For example, maybe we can have functions in configuration_test.go that we use to set-up the test environment that we also use in repositories_test.go. Does that make sense? Food for thought.

I'll work on getting a CI set-up as well.

vinc commented 11 years ago

Yes it makes perfect sense, before adding more tests we'll need to break down this one into smaller reusable components.

But there is probably an existing module built on top of testingthat is offering test fixtures like setUp and tearDown for this purpose.

pearkes commented 11 years ago

Definitely. :+1:

I'm sure that library has something in it.

pearkes commented 11 years ago

Merging the tests into master!