fix(form): preperly clean up when invalid widget is removed

Removing invalid widget sometimes resulted in improper cleanup of the form state.

1 files changed, 38 insertions, 5 deletions

diff --git a/test/ng/directive/formSpec.js b/test/ng/directive/formSpec.js
index 491a31db..3e672a7b 100644
--- a/test/ng/directive/formSpec.js
+++ b/test/ng/directive/formSpec.js
@@ -189,16 +189,16 @@ describe('form', function() {
 
     it('should deregister a child form when its DOM is removed', function() {
       doc = jqLite(
-          '<form name="parent">' +
-            '<div class="ng-form" name="child">' +
-              '<input ng:model="modelA" name="inputA" required>' +
-            '</div>' +
+        '<form name="parent">' +
+          '<div class="ng-form" name="child">' +
+          '<input ng:model="modelA" name="inputA" required>' +
+          '</div>' +
           '</form>');
       $compile(doc)(scope);
       scope.$apply();
 
       var parent = scope.parent,
-          child = scope.child;
+        child = scope.child;
 
       expect(parent).toBeDefined();
       expect(child).toBeDefined();
@@ -211,6 +211,39 @@ describe('form', function() {
     });
 
 
+    it('should deregister a input when its removed from DOM', function() {
+      doc = jqLite(
+        '<form name="parent">' +
+          '<div class="ng-form" name="child">' +
+            '<input ng:model="modelA" name="inputA" required>' +
+          '</div>' +
+        '</form>');
+      $compile(doc)(scope);
+      scope.$apply();
+
+      var parent = scope.parent,
+          child = scope.child,
+          input = child.inputA;
+
+      expect(parent).toBeDefined();
+      expect(child).toBeDefined();
+      expect(parent.$error.required).toEqual([child]);
+      expect(child.$error.required).toEqual([input]);
+      expect(doc.hasClass('ng-invalid')).toBe(true);
+      expect(doc.hasClass('ng-invalid-required')).toBe(true);
+      expect(doc.find('div').hasClass('ng-invalid')).toBe(true);
+      expect(doc.find('div').hasClass('ng-invalid-required')).toBe(true);
+      doc.find('input').remove(); //remove child
+
+      expect(parent.$error.required).toBe(false);
+      expect(child.$error.required).toBe(false);
+      expect(doc.hasClass('ng-valid')).toBe(true);
+      expect(doc.hasClass('ng-valid-required')).toBe(true);
+      expect(doc.find('div').hasClass('ng-valid')).toBe(true);
+      expect(doc.find('div').hasClass('ng-valid-required')).toBe(true);
+    });
+
+
     it('should chain nested forms in repeater', function() {
       doc = jqLite(
          '<ng:form name=parent>' +