search

brogrammers / eventhub-api

Eventhub API
1 stars 0 forks source link

Added tests to validate groups data constraints #9

Closed foFox closed 11 years ago

foFox commented 11 years ago

I have added validations to groups. We should also think about validating size of destinations , members and invited to prevent any issues due to oversized sets ( some trying to fuck with us ). Look at places where I call .reload. for some reason in some cases until I call .reload the object has its relationships cached in memory which creates an issue, I am not sure if its an error or not, and if its gonna be an issue later on.

tsov commented 11 years ago

Check this out: http://guides.rubyonrails.org/association_basics.html#controlling-caching

There is a way of reloading without invoking reload method, might look nicer? And we can use it in all the tests then

tsov commented 11 years ago

Cool man!