EdColeman commented on code in PR #3927:
URL: https://github.com/apache/accumulo/pull/3927#discussion_r1387276310
##########
core/src/test/java/org/apache/accumulo/core/conf/PropertyTest.java:
##########
@@ -105,6 +107,35 @@ public void testPorts() {
}
}
+ @Test
+ public void testJson() {
+
assertFalse(TSERV_COMPACTION_SERVICE_META_EXECUTORS.getType().isValidFormat("notJson"));
+
+ // use "real" example
+ String json1 =
+
"[{'name':'small','type':'internal','maxSize':'32M','numThreads':2},{'name':'huge','type':'internal','numThreads':2}]"
+ .replaceAll("'", "\"");
+ // use synthetic, but valid json
+ String json2 =
Review Comment:
Added another sample in 7fa8acabb7 - the test requires a single json object,
so removing the brackets allow fails.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
