Skip to content

Commit 9fc6d90

Browse files
zbobrclaude
andcommitted
fix: update queryReplySuccessTest to expect default QoS values
Remove deprecated setPriority(DATA_HIGH) and setCongestionControl(DROP) calls from queryReplySuccessTest, and update assertions to expect the default values (Priority.DATA, CongestionControl.BLOCK) since the JNI layer now ignores user-set QoS parameters on replies. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 14d90fd commit 9fc6d90

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

zenoh-java/src/jvmTest/java/io/zenoh/QueryableTest.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -137,8 +137,6 @@ public void queryReplySuccessTest() throws ZError, InterruptedException {
137137
Queryable queryable = session.declareQueryable(testKeyExpr, query -> {
138138
var options = new ReplyOptions();
139139
options.setTimeStamp(timestamp);
140-
options.setPriority(Priority.DATA_HIGH);
141-
options.setCongestionControl(CongestionControl.DROP);
142140
options.setExpress(true);
143141
try {
144142
query.reply(testKeyExpr, message, options);
@@ -158,9 +156,9 @@ public void queryReplySuccessTest() throws ZError, InterruptedException {
158156
var sample = ((Reply.Success) receivedReply[0]).getSample();
159157
assertEquals(message, sample.getPayload());
160158
assertEquals(timestamp, sample.getTimestamp());
161-
assertEquals(Priority.DATA_HIGH, sample.getPriority());
159+
assertEquals(Priority.DATA, sample.getPriority());
162160
assertTrue(sample.getQos().getExpress());
163-
assertEquals(CongestionControl.DROP, sample.getCongestionControl());
161+
assertEquals(CongestionControl.BLOCK, sample.getCongestionControl());
164162
}
165163

166164
@Test

0 commit comments

Comments
 (0)