omry / omegaconf / 60efafc6-f0cb-44fd-ae81-6125fb209011

100%

master: 100%

Pull #1258

circleci

Fix bug preserving identity on config self-assignment

Assigning an OmegaConf container node to itself used to flow through the normal Node assignment path. That path deep-copies incoming nodes before storing them, so code like cfg.foo = cfg.foo or cfg[0] = cfg[0] could replace the stored child container and leave existing references detached from the config.

Fix the bug by detecting exact self-assignment before the deepcopy/reparent step and returning early after preserving readonly enforcement. This keeps normal cross-config and cross-key node assignment deepcopy semantics unchanged while making true no-op self-assignment preserve container identity.

Add coverage for DictConfig child self-assignment, nested ListConfig element self-assignment through public API, and readonly self-assignment. Add a bugfix news fragment for issue 1177.

5 of 5 new or added lines in 1 file covered. (100.0%)

3487 of 3487 relevant lines covered (100.0%)

1.0 hits per line