added tests for ignoring hidden elements by default

hdragomir · hdragomir · commit b7b79b43a61c · 2011-09-12T11:11:15.000+02:00
diff --git a/jquery.validate.js b/jquery.validate.js
@@ -212,7 +212,7 @@ $.extend($.validator, {
 		errorContainer: $( [] ),
 		errorLabelContainer: $( [] ),
 		onsubmit: true,
-		ignore: [":hidden"],
+		ignore: ":hidden",
 		ignoreTitle: false,
 		onfocusin: function(element) {
 			this.lastActive = element;
diff --git a/test/test.js b/test/test.js
@@ -1137,3 +1137,30 @@ test("validate radio on click", function() {
 	trigger(e1);
 	errors(0);
 });
+
+test("ignore hidden elements", function(){
+    var form = $('#userForm');
+    var validate = form.validate({
+        rules:{
+            "username": "required"
+        }
+    });
+    form.get(0).reset();
+    ok(! validate.form(), "form should be initially invalid");
+    $('#userForm [name=username]').hide();
+    ok(validate.form(), "hidden elements should be ignored by default");
+});
+
+test("ignore hidden elements at start", function(){
+    var form = $('#userForm');
+    var validate = form.validate({
+        rules:{
+            "username": "required"
+        }
+    });
+    form.get(0).reset();
+    $('#userForm [name=username]').hide();
+    ok(validate.form(), "hidden elements should be ignored by default");
+    $('#userForm [name=username]').show();
+    ok(! validate.form(), "form should be invalid when required element is visible");
+});
