From 9943d9820bed1a6e9925032c56da22467be58a28 Mon Sep 17 00:00:00 2001 From: Michael Ilyin Date: Thu, 5 Mar 2026 18:35:29 +0000 Subject: [PATCH] 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 --- zenoh-java/src/jvmTest/java/io/zenoh/QueryableTest.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/zenoh-java/src/jvmTest/java/io/zenoh/QueryableTest.java b/zenoh-java/src/jvmTest/java/io/zenoh/QueryableTest.java index 8a2da00..ee3a401 100644 --- a/zenoh-java/src/jvmTest/java/io/zenoh/QueryableTest.java +++ b/zenoh-java/src/jvmTest/java/io/zenoh/QueryableTest.java @@ -137,8 +137,6 @@ public void queryReplySuccessTest() throws ZError, InterruptedException { Queryable queryable = session.declareQueryable(testKeyExpr, query -> { var options = new ReplyOptions(); options.setTimeStamp(timestamp); - options.setPriority(Priority.DATA_HIGH); - options.setCongestionControl(CongestionControl.DROP); options.setExpress(true); try { query.reply(testKeyExpr, message, options); @@ -158,9 +156,9 @@ public void queryReplySuccessTest() throws ZError, InterruptedException { var sample = ((Reply.Success) receivedReply[0]).getSample(); assertEquals(message, sample.getPayload()); assertEquals(timestamp, sample.getTimestamp()); - assertEquals(Priority.DATA_HIGH, sample.getPriority()); + assertEquals(Priority.DATA, sample.getPriority()); assertTrue(sample.getQos().getExpress()); - assertEquals(CongestionControl.DROP, sample.getCongestionControl()); + assertEquals(CongestionControl.BLOCK, sample.getCongestionControl()); } @Test