When the ReshardingCoordinatorService tries to persist its transition to kDone, it calls it tries to do so via resharding::persistStateTransitionAndCatalogUpdatesThenBumpShardVersions. However, at this point, we do not want to bump the shard version of the collection (see notifyForStateTransition) and will hit an invariant.

Our current testing does not catch this because it tests the transition to kDone by calling removeCoordinatorDocAndReshardingFields, which is a function only called by the test that no longer accurately mirrors the coordinator's code flow since SERVER-51291.