You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@storm.apache.org by sr...@apache.org on 2015/06/22 06:38:23 UTC

[3/5] storm git commit: Added in some tests

Added in some tests


Project: http://git-wip-us.apache.org/repos/asf/storm/repo
Commit: http://git-wip-us.apache.org/repos/asf/storm/commit/b5ace222
Tree: http://git-wip-us.apache.org/repos/asf/storm/tree/b5ace222
Diff: http://git-wip-us.apache.org/repos/asf/storm/diff/b5ace222

Branch: refs/heads/master
Commit: b5ace2227f403aed732374e99d3a93e9f48a5e98
Parents: 323fea2
Author: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Authored: Tue May 5 10:17:46 2015 -0500
Committer: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Committed: Tue May 5 10:17:46 2015 -0500

----------------------------------------------------------------------
 storm-core/test/clj/backtype/storm/config_test.clj | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/storm/blob/b5ace222/storm-core/test/clj/backtype/storm/config_test.clj
----------------------------------------------------------------------
diff --git a/storm-core/test/clj/backtype/storm/config_test.clj b/storm-core/test/clj/backtype/storm/config_test.clj
index a7498ba..9750185 100644
--- a/storm-core/test/clj/backtype/storm/config_test.clj
+++ b/storm-core/test/clj/backtype/storm/config_test.clj
@@ -74,6 +74,20 @@
     (is (thrown-cause? java.lang.IllegalArgumentException
           (.validateField validator "test" (inc Integer/MAX_VALUE))))))
 
+(deftest test-pos-integer-validator
+  (let [validator ConfigValidation/NotNullPosIntegerValidator]
+    (is (thrown-cause? java.lang.IllegalArgumentException
+        (.validateField validator "test" nil)))
+    (.validateField validator "test" 1000)
+    (is (thrown-cause? java.lang.IllegalArgumentException
+          (.validateField validator "test" 1.34)))
+    (is (thrown-cause? java.lang.IllegalArgumentException
+          (.validateField validator "test" 0)))
+    (is (thrown-cause? java.lang.IllegalArgumentException
+          (.validateField validator "test" -100)))
+    (is (thrown-cause? java.lang.IllegalArgumentException
+          (.validateField validator "test" (inc Integer/MAX_VALUE))))))
+
 (deftest test-integers-validator
   (let [validator ConfigValidation/IntegersValidator]
     (.validateField validator "test" nil)