Fixed the on-demand loading of animation and added necessary unit tes…

…t. This closes #31

README.md

@@ -97,9 +97,7 @@ Here is the list of methods supported by `app.register`:
 * `.value`
 * `.directive`
 * `.filter`
-* `.animation` **
-
-** `.animation` is not covered in unit test for now
+* `.animation`
 
 ### 3rd Party AngularJS Modules
 

bower.json

@@ -2,8 +2,10 @@
   "name": "angularAMD",
   "version": "0.1.0",
   "dependencies": {
-    "angular": "~1.2.16",
-    "angular-route": "~1.2.16",
+    "angular": "1.2.16",
+    "angular-route": "1.2.16",
+    "angular-animate": "1.2.16",
+    "angular-mocks": "1.2.16",
     "angular-ui-bootstrap-bower": "~0.6.0",
     "requirejs": "~2.1.9",
     "requirejs-plugins": "~1.0.2",

src/angularAMD.js

@@ -281,7 +281,7 @@ define(function () {
                     service: provide.service,
                     constant: provide.constant,
                     value: provide.value,
-                    animation: animateProvider.register
+                    animation: angular.bind(animateProvider, animateProvider.register)
                 };
 
             }]

test/conf/karma.unit.mustache

@@ -12,7 +12,7 @@ module.exports = function (config) {
         
         // list of files / patterns to load in the browser
         files: [
-            {pattern: 'bower_components/angular/*.js', watched: false, included: false},
+            {pattern: 'bower_components/**/*.js', watched: false, included: false},
             {pattern: 'build/*.js', watched: true, included: false},
             {pattern: 'src/*.js', watched: true, included: false},
             {pattern: 'test/unit/*.spec.js', watched: true, included: false},

test/unit/lib/app.js

@@ -1,22 +1,23 @@
 /*jslint browser: true, devel: true, node: true, vars: true, nomen: true */
 /*globals define, angular */
 
-define(['angularAMD'], function (angularAMD) {
+define(['angularAMD', 'angular-animate'], function (angularAMD) {
     'use strict';
 
     /**
      * BOOTSTRAP ANGULAR
      * Replicating what would normally take place in app.js
      */
     var app_name = "unitest-app",
-        app = angular.module(app_name, []);
+        app = angular.module(app_name, ['ngAnimate']);
 
     // Add property for unit test
     app.__appname = app_name;
     app.__origAngular = window.angular;
 
+
     /*
-    var elem = document.createElement('div');
+    var elem = document.body;
     angularAMD.bootstrap(app, undefined, elem);
     */
     angularAMD.bootstrap(app);

test/unit/lib/main.mustache

@@ -7,6 +7,7 @@ require.config({
     // alias libraries paths
     paths: {
         'angular': 'bower_components/angular/angular',
+        'angular-animate': 'bower_components/angular-animate/angular-animate',
         'angular-mocks': 'bower_components/angular-mocks/angular-mocks',
         'angularAMD': '{{{angularAMD-js-file}}}',
         'ngload': '{{{ngload-js-file}}}',
@@ -20,6 +21,9 @@ require.config({
 
     shim: {
         'angularAMD': ['angular'],
+        'angular-animate': ['angular'],
+        'angular-mocks': ['angular'],
+        
         /*
         'services' in this case is a regular angular.js module and therefore non-requirejs module so 
         must be defined in shim.  This is critical as 'services' depends on processing in 'app' to

test/unit/lib/regServices.js

@@ -50,6 +50,24 @@ define(['app', 'angularAMD'], function(app, angularAMD) {
         };
     });
 
+    // Create Animation
+    services.animation('.service-reg-animation', function ($log, $interval) {
+        return {
+            addClass : function(element, className, done) {
+                if ( className === "custom-hide") {
+                    element.css('opacity',0);
+                    done();
+                }
+            },
+            removeClass : function(element, className, done) {
+                if ( className === "custom-hide") {
+                    element.css('opacity',1);
+                    done();
+                }
+            }
+        }
+    });
+
     // Return the result in a factory
     services.factory('UtestRegServiceResult', function () {
         return utest_reg_result;

test/unit/regServices.spec.js

@@ -4,12 +4,12 @@
 /**
  * Test to ensure that services can be created using app.register method.
  */
-define(['regServices','angularAMD'], function (app, angularAMD) {
+define(['regServices','angularAMD', 'angular-mocks'], function (app, angularAMD) {
     'use strict';
+    var inject = angularAMD.inject;
     describe('Utest Registered Services', function () {
         //console.log("Running serviceSpec.js");
-        var results = app.__utest_regserv_result,
-            inject = angularAMD.inject;
+        var results = app.__utest_regserv_result;
 
         it(".reg_constant check.", function () {
             inject(function (reg_constant_name) {
@@ -55,7 +55,31 @@ define(['regServices','angularAMD'], function (app, angularAMD) {
                 expect(ufilter("hello")).toBe("hello " + results.reg_filter_name);
             });
         });
-
+    });
+
+    describe("Utest Registered Animation", function () {
+        var scope, animate, elem;
 
+        beforeEach(function () {
+            module("ngAnimateMock");
+            inject(function ($rootScope, $compile, $rootElement, $animate) {
+                var html_text = "<div ng-class='AnimationClass' class='service-reg-animation'></div>";
+                scope = $rootScope;
+                animate = $animate;
+                elem = $compile(html_text)(scope);
+                $rootElement.append(elem);
+            });
+        });
+
+        it(".animation check.", function () {
+            animate.addClass(elem, "custom-hide");           
+            scope.$digest();
+            expect(elem.css("opacity")).toBe("0");
+
+            animate.removeClass(elem, "custom-hide");
+            scope.$digest();
+            expect(elem.css("opacity")).toBe("1");
+        });
     });
+
 });