fix(popover): extend options to modal service

Closes #2724

Author: adamdbradley

js/angular/service/popover.js

@@ -212,8 +212,8 @@ function($ionicModal, $ionicPosition, $document, $window) {
      * @returns {promise} A promise that will be resolved with an instance of
      * an {@link ionic.controller:ionicPopover} controller (ionicPopover is built on top of $ionicPopover).
      */
-    fromTemplateUrl: function(url, options, _) {
-      return $ionicModal.fromTemplateUrl(url, options, ionic.Utils.extend(POPOVER_OPTIONS, options || {}));
+    fromTemplateUrl: function(url, options) {
+      return $ionicModal.fromTemplateUrl(url, ionic.Utils.extend(POPOVER_OPTIONS, options || {}));
     }
   };
 

test/unit/angular/service/popover.unit.js

@@ -36,7 +36,7 @@ describe('Ionic Popover', function() {
 
     var done = false;
 
-    var instance = popover.fromTemplateUrl('popover.html', function(instance) {
+    popover.fromTemplateUrl('popover.html').then(function(instance) {
       done = true;
       instance.show();
       expect(instance.el.classList.contains('popover-backdrop')).toBe(true);