test: switch from should style to expect

Switches from using `should` assertions (e.g. `foo.should.equal(303)`)
to `expect` assertions (e.g. `expect(foo).to.equal(303)`).

This is a stepping stone to moving off `sinon` and possibly `chai` in
future.