Added test coverage...

[ztbrown]

for everything except for the application controller. I think the application controller needs to be decoupled from the 'cola/dom/form' injection to be tested in isolation.

[unscriptable]

Rock on @ztbrown015! This looks great!

Just one minor quibble: rename test/app/contacts/cleanContacts-test.js to test/app/contacts/cleanContact-test.js (remove "s" on end of cleanContact to match original file).

Pinging @briancavalier and @habuma to see if they have any objections to merging this.

-- John

[briancavalier]

Very cool, @ztbrown015, thanks!

Looks good to me, aside from @unscriptable's comment. I'll let @habuma have a look as well, and hopefully we can get this merged soon.

[ztbrown]

Thanks for the reviewing this! I saw @habuma present on cujo in Columbus this weekend and am really excited to use it in a future project.

[briancavalier]

@ztbrown015 great! If you have questions, or want to chat realtime about cujoJS, feel free to join us in #cujojs on freenode IRC.

[briancavalier]

I can't imagine @habuma will object :) I added some very minor comments. Once we work through those, I'll go ahead and merge this!

[briancavalier]

Also, I have a change that will remove the controller's hard dependency on cola/dom/form as @ztbrown015 rightly suggested. It'll def help with the controller unit testing situation. After we get this merged, I'll rebase and pull request my change.

[briancavalier]

Looks great. Thanks for the update.