search

jakdept / moxxi

A proxy to transparently rewrite requests and send them to a destination server.
MIT License
1 stars 0 forks source link

moxxiConf package needs more tests #3

Open jakdept opened 8 years ago

jakdept commented 8 years ago

shooting for 100% test coverage

jakdept commented 8 years ago

[x] pulled in a branch that added lots of testing for writer.go - not 100% but close. [x] have 100% coverage for type.go [ ] need to start/work on coverage for handler.go

jakdept commented 8 years ago

added config.go and it doesn't have full test coverage.

basically, there's more files now, and more testing to be done, but my testing % has dropped over time.

will need to focus on this in the future.

jakdept commented 8 years ago

really, at this point, I need to write handler_test.go to get 100% coverage of handler.go

I'm thinking of skipping going for testing coverage of config.go because i want to write a new config system (like spf13/viper but something that works) and use that instead.

jakdept commented 8 years ago

i need to add a test for the json handler yet (not 100% coverage) then i'll have everything but the config.

i'll call it good after that.