fix(ngAnimate): ensure SVG classes are properly removed

matsko · matsko · commit fa0bbded1ea0 · 2015-04-10T13:32:27.000-07:00
diff --git a/src/ngAnimate/shared.js b/src/ngAnimate/shared.js
@@ -110,7 +110,6 @@ function applyAnimationClassesFactory($$jqLite) {
     }
     if (options.removeClass) {
       $$removeClass($$jqLite, element, options.removeClass);
-      element.removeClass(options.removeClass);
       options.removeClass = null;
     }
   }
diff --git a/test/ngAnimate/animateCssSpec.js b/test/ngAnimate/animateCssSpec.js
@@ -2421,7 +2421,8 @@ describe("ngAnimate $animateCss", function() {
           duration: 0
         });
 
-        expect(child.attr('class')).toBe('');
+        var className = child[0].getAttribute('class');
+        expect(className).toBe('');
       }));
     });
   });

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,6 @@ function applyAnimationClassesFactory($$jqLite) {`
`110`	`110`	`}`
`111`	`111`	`if (options.removeClass) {`
`112`	`112`	`$$removeClass($$jqLite, element, options.removeClass);`
`113`		`- element.removeClass(options.removeClass);`
`114`	`113`	`options.removeClass = null;`
`115`	`114`	`}`
`116`	`115`	`}`