ovn-kubernetes / libovsdb / 25868380124

50%

main: 48%

Build Type

Pull #472

github

Committed by dave-tucker

Commit Message

test: fix TestMultipleOpsTransactIntegration map merge bug

model.CloneInto uses json.Unmarshal which merges into existing maps
rather than replacing them. If the Eventually loop runs once before
the update3 modify notification is processed, br.ExternalIDs gets
set to the pre-mutation state {docker, go}. On subsequent iterations
json.Unmarshal merges the updated state into the existing map, so
docker is never removed and the condition is never satisfied.

Fix by recreating br on each iteration so json.Unmarshal always
starts from a nil map and produces a clean result.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>

Pull Request Pull Request #472: client: introduce ConnectionManager to fix reconnect/disconnect panics

Coverage Stats

815 of 1014 new or added lines in 5 files covered. (80.37%)

17 existing lines in 1 file now uncovered.

6537 of 13271 relevant lines covered (49.26%)

117.07 hits per line