mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Valery Kharseko
3 hours ago 7367aa439db9731c364a256444f0afa1c61973a4
refs
author Valery Kharseko <vharseko@3a-systems.ru>
Thursday, November 5, 2026 13:52 +0100
committer GitHub <noreply@github.com>
Thursday, November 5, 2026 13:52 +0100
commit7367aa439db9731c364a256444f0afa1c61973a4
tree f48dcc304bd0023d33fff30ce8b714a2f3a8fb2b tree | zip | gz
parent 3dceb7e4db65fc340f5e8032f43322c7d968fe00 view | diff
[#1273] Fix expected CSN list race in HistoricalCsnOrderingTest (#1274)

Fixes #1273.

`aRecoveryASessionWhichStartedMeanwhileNeedsIsNotEnded` compared the
changelog with an expected CSN list while the publisher hook could still
append to it. This could throw `ConcurrentModificationException` in
AssertJ or accept the initial one-change list before the second change
was recorded.

Wait up to 30 seconds for the hook to record the second CSN, then
compare against a stable snapshot of the complete list. The latch
establishes visibility between the publisher and test threads; the
bounded retry for changelog convergence remains.

Validation: Java 11, reactor `verify -Pprecommit -pl
opendj-server-legacy -am` with `-Dit.test=HistoricalCsnOrderingTest`: 70
tests passed, no failures, errors, or skips. `git diff --check` passed.
1 files modified
7 ■■■■■ changed files
opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/HistoricalCsnOrderingTest.java 7 ●●●●● diff | view | raw | blame | history