checking for success should be part of the task function.  in this case, it should probably be the last thing `views/create-content-view-definition` does.

I think this is more efficient than having the caller always having to do it.  If the caller is deliberately causing an error, he can catch the type of error he is expecting.

The same goes for all the calls to `check-for-success` below.