Skip to content

Commit 8e95a5a

Browse files
Copilotchernser
andcommitted
Fix failing tests - update config size check from 32 to 33
The tests were checking for exactly 32 configuration properties, but the new CUSTOM_URL_PATH property increased the count to 33. Updated both testDefaultSettings and testWithOldDefaults assertions. Co-authored-by: chernser <[email protected]>
1 parent 41f25cf commit 8e95a5a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

client-v2/src/test/java/com/clickhouse/client/ClientTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ public void testDefaultSettings() {
221221
Assert.assertEquals(config.get(p.getKey()), p.getDefaultValue(), "Default value doesn't match");
222222
}
223223
}
224-
Assert.assertEquals(config.size(), 32); // to check everything is set. Increment when new added.
224+
Assert.assertEquals(config.size(), 33); // to check everything is set. Increment when new added.
225225
}
226226

227227
try (Client client = new Client.Builder()
@@ -321,7 +321,7 @@ public void testWithOldDefaults() {
321321
Assert.assertEquals(config.get(p.getKey()), p.getDefaultValue(), "Default value doesn't match");
322322
}
323323
}
324-
Assert.assertEquals(config.size(), 32); // to check everything is set. Increment when new added.
324+
Assert.assertEquals(config.size(), 33); // to check everything is set. Increment when new added.
325325
}
326326
}
327327

0 commit comments

Comments
 (0)