should we add ```Assert.fail()``` here ? (e.g. if ```doThrow``` is commented out, by accident)