fix(ngAnimate): ensure that a filtered-out leave animation always run…

…s its DOM operation

This patch fixes the issue where filtered-out leave animations were not
properly run the DOM operation when closed.

Closes #11555

src/ngAnimate/animateQueue.js

@@ -250,7 +250,7 @@ var $$AnimateQueueProvider = ['$animateProvider', function($animateProvider) {
 
       var className = [node.className, options.addClass, options.removeClass].join(' ');
       if (!isAnimatableClassName(className)) {
-        runner.end();
+        close();
         return runner;
       }
 

test/ngAnimate/animateSpec.js

@@ -133,7 +133,7 @@ describe("animations", function() {
       module(function($animateProvider) {
         $animateProvider.classNameFilter(/only-allow-this-animation/);
       });
-      inject(function($animate, $rootScope, $document, $rootElement) {
+      inject(function($animate, $rootScope) {
         expect(element).not.toHaveClass('only-allow-this-animation');
 
         $animate.enter(element, parent);
@@ -148,6 +148,22 @@ describe("animations", function() {
       });
     });
 
+    it('should complete the leave DOM operation in case the classNameFilter fails', function() {
+      module(function($animateProvider) {
+        $animateProvider.classNameFilter(/memorable-animation/);
+      });
+      inject(function($animate, $rootScope) {
+        expect(element).not.toHaveClass('memorable-animation');
+
+        parent.append(element);
+        $animate.leave(element);
+        $rootScope.$digest();
+
+        expect(capturedAnimation).toBeFalsy();
+        expect(element[0].parentNode).toBeFalsy();
+      });
+    });
+
     describe('enabled()', function() {
       it("should work for all animations", inject(function($animate) {