omry / omegaconf / 63725883-6024-4b74-8dea-c38b035861b0

100%

push

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%)

3494 of 3494 relevant lines covered (100.0%)

1.0 hits per line