This method needs a `yield`...so the spec below isn't testing anything.

It's a good idea to "break" the implementation and see the test fail when you refactor it like this to confirm it still fails with a good failure message in the way you expect.