Promises in tests are definitely to be desired. We can remove done and pretty much all `expect(err).to.not.be.ok` as @janmeier  says