fix($animate): improve detection on ng-animate in classNameFilter RegExp

Author: gkalpak

src/ng/animate.js

@@ -179,6 +179,7 @@ var $$CoreAnimateQueueProvider = /** @this */ function() {
  */
 var $AnimateProvider = ['$provide', /** @this */ function($provide) {
   var provider = this;
+  var classNameFilter = null;
 
   this.$$registeredAnimations = Object.create(null);
 
@@ -247,15 +248,15 @@ var $AnimateProvider = ['$provide', /** @this */ function($provide) {
    */
   this.classNameFilter = function(expression) {
     if (arguments.length === 1) {
-      this.$$classNameFilter = (expression instanceof RegExp) ? expression : null;
-      if (this.$$classNameFilter) {
-        var reservedRegex = new RegExp('(\\s+|\\/)' + NG_ANIMATE_CLASSNAME + '(\\s+|\\/)');
-        if (reservedRegex.test(this.$$classNameFilter.toString())) {
-          throw $animateMinErr('nongcls','$animateProvider.classNameFilter(regex) prohibits accepting a regex value which matches/contains the "{0}" CSS class.', NG_ANIMATE_CLASSNAME);
+      classNameFilter = (expression instanceof RegExp) ? expression : null;
+      if (classNameFilter) {
+        var reservedRegex = new RegExp('[(\\s|\\/)]' + NG_ANIMATE_CLASSNAME + '[(\\s|\\/)]');
+        if (reservedRegex.test(classNameFilter.toString())) {
+          throw $animateMinErr('nongcls', '$animateProvider.classNameFilter(regex) prohibits accepting a regex value which matches/contains the "{0}" CSS class.', NG_ANIMATE_CLASSNAME);
         }
       }
     }
-    return this.$$classNameFilter;
+    return classNameFilter;
   };
 
   this.$get = ['$$animateQueue', function($$animateQueue) {

test/ngAnimate/animateSpec.js

@@ -255,29 +255,33 @@ describe('animations', function() {
       });
     });
 
-    it('should throw a minErr if a regex value is used which partially contains or fully matches the `ng-animate` CSS class', function() {
+    it('should throw a minErr if a regex value is used which partially contains or fully matches the `ng-animate` CSS class',
       module(function($animateProvider) {
+        var errorMessage = '$animateProvider.classNameFilter(regex) prohibits accepting a regex ' +
+                           'value which matches/contains the "ng-animate" CSS class.';
+
         assertError(/ng-animate/, true);
         assertError(/first ng-animate last/, true);
         assertError(/ng-animate-special/, false);
         assertError(/first ng-animate-special last/, false);
         assertError(/first ng-animate ng-animate-special last/, true);
+        assertError(/(ng-animate)/, true);
+        assertError(/(foo|ng-animate|bar)/, true);
+        assertError(/(foo|)ng-animate(|bar)/, true);
 
         function assertError(regex, bool) {
           var expectation = expect(function() {
             $animateProvider.classNameFilter(regex);
           });
 
-          var message = '$animateProvider.classNameFilter(regex) prohibits accepting a regex value which matches/contains the "ng-animate" CSS class.';
-
           if (bool) {
-            expectation.toThrowMinErr('$animate', 'nongcls', message);
+            expectation.toThrowMinErr('$animate', 'nongcls', errorMessage);
           } else {
-            expectation.not.toThrowMinErr('$animate', 'nongcls', message);
+            expectation.not.toThrow();
           }
         }
-      });
-    });
+      })
+    );
 
     it('should complete the leave DOM operation in case the classNameFilter fails', function() {
       module(function($animateProvider) {