fix(macPopover): Fix mac-popover-refresh-on not registering properly

Fixes #38
angular-macgyver · Sep 13, 2016 · ec9e4ff · ec9e4ff
1 parent 2ef5ce8
commit ec9e4ff
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/src/constants.js b/src/constants.js
@@ -53,7 +53,8 @@ constant('macPopoverDefaults', {
     footer: false,
     header: false,
     title: '',
-    direction: 'above left'
+    direction: 'above left',
+    refreshOn: ''
   },
   template: "<div class='mac-popover' ng-class='macPopoverClasses'>\
     <div class='tip'></div>\

diff --git a/test/unit/popover.spec.js b/test/unit/popover.spec.js
@@ -12,8 +12,7 @@ describe("Popover directive", function() {
     var element;
 
     beforeEach(function () {
-      var template;
-      template = "<div><mac-popover id='testPopover'>Test</mac-popover></div>";
+      var template = "<div><mac-popover id='testPopover'>Test</mac-popover></div>";
       element = $compile(template)($rootScope);
       $rootScope.$digest();
     });
@@ -49,6 +48,18 @@ describe("Popover directive", function() {
     });
   });
 
+  describe("popover element register", function() {
+    it('should register with refreshOn options', function() {
+      var template = "<div><mac-popover id='testPopover' mac-popover-refresh-on='testEvent'>Test</mac-popover></div>";
+      $compile(template)($rootScope);
+      $rootScope.$digest();
+
+      var testPopoverOptions = popover.registered.testPopover
+      expect(testPopoverOptions).toBeDefined();
+      expect(testPopoverOptions.refreshOn).toBe('testEvent');
+    });
+  });
+
   describe("popover trigger", function () {
     var trigger, $timeout;