Adding the public "updateState" method is imho a mistake, that opens the possibility to change the state in (in fact) impossible ways like from closed to resolved or from resolved to open (instead of reopen). Introducing this method destroys all encapsulation and validation for invariants (and the idea of a workflow in general).