search

pjasiun / fuma

Fussball Manager for Slack
MIT License
4 stars 4 forks source link

Don't allow player to be registered twice. #29

Closed jodator closed 8 years ago

jodator commented 8 years ago

Don't allow player to be registered twice.

jodator commented 8 years ago

OK, I'll fix that later on. I was doing this with my morning coffee so I've missed that :D

jodator commented 8 years ago

OK, this must be merged. I've added tests & this command have 100% coverage. For this to work I've revamped main handler a bit and moved asyncResponse there (this will be preferable way of passing asyn answer as it is easier to write tests (Inversion of Control). After that is merged I can work on tests for other commands :)