Reviewed Additional Javascript Tests

@clarkbarz reviewed the javascript tests in pull request #128, which was requested in #127. 
I think this is the wrong way to test this.

Instead of testing that destroy, closeAll, and shelf.close were called, we should make sure that they actually work.

you can make sure that the node is no longer in the collection, that the node_view_list no longer has subviews.  ShelfView doesn't define it's own close method, so maybe just checking that it was called is fine.

But just checking that these methods are called is not enough.