My take is that the assertions inside the catch block should just be assertions about the exception.

I prefer the `fail()` to be next to the line of code that we expected to throw the exception, rather than after the try-catch as I think that is clearer what the failure is associated with.