You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by ghajos <gi...@git.apache.org> on 2018/04/02 11:17:03 UTC
[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...
Github user ghajos commented on a diff in the pull request:
https://github.com/apache/storm/pull/2583#discussion_r178527233
--- Diff: storm-client/test/jvm/org/apache/storm/utils/UtilsTest.java ---
@@ -173,4 +176,49 @@ public void isZkAuthenticationConfiguredStormServerWithPropertyTest() {
}
}
}
+
+ @Test
--- End diff --
Found a positive test case in TestConfigValidate and added negative test cases. Should I collect some more test cases?
---