search

brogrammers / eventhub-api

Eventhub API
1 stars 0 forks source link

Adding groups controller #34

Open foFox opened 11 years ago

foFox commented 11 years ago

I added basic REST methods do groups controller

tsov commented 11 years ago

Sweet! Is this updated? Sorry for all those stupid comments of mine

tsov commented 11 years ago

Yo, how far is this?

foFox commented 11 years ago

I think the test are bit shit, I think I got lost on how to test this, as there's so many possibilities, if you want we can remove then and start again.

tsov commented 11 years ago

There are not shit, but very extensive.. Thats what happens if you do everything at once :D 28 commits, >2000 lines added. The tests seems more like regression tests, and look nothing like the tests we have. I guess one can get very into testing everything, but i am not sure what to do here

tsov commented 11 years ago

Otherwise, how far are you on this?

foFox commented 11 years ago

We can scrap them if you want :P