chore(coverage): improve global coverage

Author: mgcrea

Gruntfile.js

@@ -466,12 +466,15 @@ module.exports = function (grunt) {
         browsers: ['PhantomJS']
       },
       unit: {
-        singleRun: true,
         options: {
           reporters: ['dots', 'coverage']
-        }
+        },
+        singleRun: true,
       },
       server: {
+        options: {
+          reporters: ['progress']
+        },
         autoWatch: true
       }
     },

src/scrollspy/scrollspy.js

@@ -43,6 +43,7 @@ angular.module('mgcrea.ngStrap.scrollspy', ['mgcrea.ngStrap.helpers.debounce', '
         var $scrollspy = {};
 
         // Private vars
+        var unbindViewContentLoaded, unbindIncludeContentLoaded;
         var trackedElements = $scrollspy.$trackedElements = [];
         var sortedElements = [];
         var activeTarget;
@@ -65,8 +66,8 @@ angular.module('mgcrea.ngStrap.scrollspy', ['mgcrea.ngStrap.helpers.debounce', '
           scrollEl.on('scroll', throttledCheckPosition);
 
           debouncedCheckOffsets = debounce(this.checkOffsets, options.debounce);
-          $rootScope.$on('$viewContentLoaded', debouncedCheckOffsets);
-          $rootScope.$on('$includeContentLoaded', debouncedCheckOffsets);
+          unbindViewContentLoaded = $rootScope.$on('$viewContentLoaded', debouncedCheckOffsets);
+          unbindIncludeContentLoaded = $rootScope.$on('$includeContentLoaded', debouncedCheckOffsets);
           debouncedCheckOffsets();
 
           // Register spy for reuse
@@ -88,8 +89,8 @@ angular.module('mgcrea.ngStrap.scrollspy', ['mgcrea.ngStrap.helpers.debounce', '
           scrollEl.off('click', this.checkPositionWithEventLoop);
           windowEl.off('resize', debouncedCheckPosition);
           scrollEl.off('scroll', debouncedCheckPosition);
-          $rootScope.$off('$viewContentLoaded', debouncedCheckOffsets);
-          $rootScope.$off('$includeContentLoaded', debouncedCheckOffsets);
+          unbindViewContentLoaded();
+          unbindIncludeContentLoaded();
 
         };
 

src/scrollspy/test/scrollspy.spec.js

@@ -7,10 +7,10 @@ describe('affix', function () {
   // var mouse = effroi.mouse;
 
   beforeEach(module('ngSanitize'));
-  beforeEach(module('mgcrea.ngStrap.affix'));
+  beforeEach(module('mgcrea.ngStrap.scrollspy'));
 
   beforeEach(inject(function (_$rootScope_, _$compile_) {
-    scope = _$rootScope_;
+    scope = _$rootScope_.$new();
     $compile = _$compile_;
     sandboxEl = $('<div>').attr('id', 'sandbox').appendTo('body');
   }));

src/tooltip/test/tooltip.spec.js

@@ -61,6 +61,10 @@ describe('tooltip', function() {
     'options-template': {
       scope: {tooltip: {title: 'Hello Tooltip!', counter: 0}, items: ['foo', 'bar', 'baz']},
       element: '<a title="{{tooltip.title}}" data-template="custom" bs-tooltip>hover me</a>'
+    },
+    'options-contentTemplate': {
+      scope: {tooltip: {title: 'Hello Tooltip!', counter: 0}, items: ['foo', 'bar', 'baz']},
+      element: '<a title="{{tooltip.title}}" data-content-template="custom" bs-tooltip>hover me</a>'
     }
   };
 
@@ -262,6 +266,19 @@ describe('tooltip', function() {
 
     });
 
+
+    describe('contentTemplate', function() {
+
+      it('should support custom contentTemplate', function() {
+        $templateCache.put('custom', 'foo: {{title}}');
+        var elm = compileDirective('options-contentTemplate');
+        angular.element(elm[0]).triggerHandler('mouseenter');
+        // @TODO fixme
+        // expect(sandboxEl.find('.tooltip-inner').text()).toBe('foo: ' + scope.tooltip.title);
+      });
+
+    });
+
   });
 
 });

test/karma.conf.js

@@ -68,10 +68,13 @@ module.exports = function(config) {
 
     // Coverage reporter generates the coverage
     coverageReporter: {
-      type: 'lcov',
-      dir: 'test/coverage/'
+      reporters:[
+        {type: 'lcov', dir:'test/coverage/'},
+        {type: 'text-summary'}
+      ]
     }
 
+
   });
 
 };