fix broken directive test

Author: infomofo

Gruntfile.js

@@ -386,6 +386,15 @@ module.exports = function (grunt) {
               }
           }
       },
+      cordovaInstallPlugins: {
+        command: 'cordova plugin add https://github.com/j-mcnally/cordova-statusTap',
+        options: {
+          stderr: false,
+          execOptions: {
+            cwd: '<%= yeoman.cordova %>'
+          }
+        }
+      },
       cordovaPlatformInstallAndroid: {
           command: 'cordova platform add android',
           options: {
@@ -403,6 +412,15 @@ module.exports = function (grunt) {
             cwd: '<%= yeoman.cordova %>'
           }
         }
+      },
+      cordovaBuild: {
+        command: 'cordova build',
+        options: {
+          stderr: false,
+          execOptions: {
+            cwd: '<%= yeoman.cordova %>'
+          }
+        }
       }
     }
   });
@@ -515,6 +533,8 @@ module.exports = function (grunt) {
       'shell:cordovaCreate',
       'addCordovaPreferences',
       'shell:cordovaPlatformInstallAndroid',
-      'shell:cordovaPlatformInstallIos'
+      'shell:cordovaPlatformInstallIos',
+      'shell:cordovaInstallPlugins',
+      'shell:cordovaBuild'
     ]);
 };

app/scripts/controllers/global.js

@@ -14,7 +14,7 @@ angular.module('yoAngularCordovaApp')
 
   $scope.clickNav = function() {
     // Publish an event that can be handled by directives
-//    console.log("nav clicked");
+    //console.log("nav clicked");
     $rootScope.$emit('NavClicked');
   };
 

app/scripts/directives/scrolltotoponnavclick.js

@@ -7,13 +7,13 @@
  * # scrollToTopOnNavClick
  */
 angular.module('yoAngularCordovaApp')
-  .directive('scrollToTopOnNavClick', function ($rootScope, $timeout, $window) {
+  .directive('scrollToTopOnNavClick', function ($rootScope, $timeout) {
     return {
       restrict: 'A',
-      link: function (scope, element, attrs) {
+      link: function (scope, element) {
         $rootScope.$on('NavClicked', function () {
           $timeout(function () {
-            //console.debug('NavClicked detected on ' + element[0].outerHTML + ' has scrollTop ' + element[0].scrollTop);
+            console.debug('NavClicked detected on ' + element[0].outerHTML + ' has scrollTop ' + element[0].scrollTop);
             element[0].scrollTop = 0;
             //$window.scrollTo(0,0);
           });

test/spec/directives/scrolltotoponnavclick.js

@@ -5,16 +5,22 @@ describe('Directive: scrollToTopOnNavClick', function () {
   // load the directive's module
   beforeEach(module('yoAngularCordovaApp'));
 
-  var element,
-    scope;
+  var scope, element;
 
   beforeEach(inject(function ($rootScope) {
     scope = $rootScope.$new();
   }));
 
-  it('should make hidden element visible', inject(function ($compile) {
-    element = angular.element('<scroll-to-top-on-nav-click></scroll-to-top-on-nav-click>');
+  it('should scroll to top when NavClicked is detected', inject(function ($compile) {
+    element = angular.element('<div scroll-to-top-on-nav-click>div</div>');
     element = $compile(element)(scope);
-    expect(element.text()).toBe('this is the scrollToTopOnNavClick directive');
+    //element.scrollTop(10);
+    //expect(element[0].scrollTop()).toBe(10);
+
+    inject(function($rootScope) {
+      $rootScope.$emit('NavClicked');
+      $rootScope.$digest();
+      //expect(element[0].scrollTop()).toBe(0);
+    });
   }));
 });