`}).then(null, assert.fail);` should be enough; is there any specific reason because you added the redundant `assert.fail`? If there is, probably having a comment would be nice, here.