Skip to content

Commit a25a31f

Browse files
committed
Enforce lazy initialization on queryStreamStats
Fixes #368
1 parent 2b5c0de commit a25a31f

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

Diff for: src/main/java/com/rabbitmq/stream/impl/StreamEnvironment.java

+1
Original file line numberDiff line numberDiff line change
@@ -480,6 +480,7 @@ public void deleteStream(String stream) {
480480
@Override
481481
public StreamStats queryStreamStats(String stream) {
482482
checkNotClosed();
483+
this.maybeInitializeLocator();
483484
StreamStatsResponse response =
484485
locatorOperation(
485486
Utils.namedFunction(

Diff for: src/test/java/com/rabbitmq/stream/impl/StreamEnvironmentTest.java

+5-3
Original file line numberDiff line numberDiff line change
@@ -552,10 +552,12 @@ void createPublishConsumeDelete(boolean lazyInit, TestInfo info) {
552552
}
553553
}
554554

555-
@Test
555+
@ParameterizedTest
556+
@ValueSource(booleans = {true, false})
556557
@BrokerVersionAtLeast(BrokerVersion.RABBITMQ_3_11)
557-
void queryStreamStatsShouldReturnFirstOffsetAndCommittedOffset() throws Exception {
558-
try (Environment env = environmentBuilder.build()) {
558+
void queryStreamStatsShouldReturnFirstOffsetAndCommittedOffset(boolean lazyInit)
559+
throws Exception {
560+
try (Environment env = environmentBuilder.lazyInitialization(lazyInit).build()) {
559561
StreamStats stats = env.queryStreamStats(stream);
560562
assertThatThrownBy(() -> stats.firstOffset()).isInstanceOf(NoOffsetException.class);
561563
assertThatThrownBy(() -> stats.committedChunkId()).isInstanceOf(NoOffsetException.class);

0 commit comments

Comments
 (0)