Remove overuse of done test callback

Currently we use done during the tests in a number of places, instead change to use chai's throw. http://chaijs.com/api/bdd/#method_throw
This will allow us to check for expected errors vs unexpected errors.
Since almost all publish calls are sync, done is unnecessary.