fix: remove service to only keep provider

Author: toutpt

src/index.js

@@ -1,4 +1,3 @@
-import service from './leaflet.service';
 import provider from './leaflet.provider';
 import controller from './leaflet.controller';
 
@@ -9,5 +8,4 @@ export default angular.module('angular-leaflet', [
     bindings: {
         onMapInitialized: '&'
     }
-}).provider('leafletService', provider)
-.service('leafletService', service);
+}).provider('leafletService', provider);

src/leaflet.provider.js

@@ -3,9 +3,9 @@ import LeafletService from './leaflet.service';
 function provider() {
     var defaultSettings = {
         tiles: {
-            url: "http://{s}.tile.openstreetmap.fr/osmfr/{z}/{x}/{y}.png",
+            url: 'http://{s}.tile.openstreetmap.fr/osmfr/{z}/{x}/{y}.png',
             options: {
-                attribution: "&copy; Openstreetmap France | &copy; <a href=\"http://www.openstreetmap.org/copyright\">OpenStreetMap</a>",
+                attribution: '&copy; Openstreetmap France | &copy; <a href="http://www.openstreetmap.org/copyright">OpenStreetMap</a>',
                 maxZoom: 19
             }
         },

src/leaflet.service.spec.js

@@ -0,0 +1,66 @@
+'use strict';
+/*global ngDescribe:false */
+/*global it:false */
+/*global expect:false */
+
+
+ngDescribe({
+    modules: 'angular-leaflet',
+    inject: ['leafletService'],
+    tests: function (deps) {
+        it('should have default settings', function() {
+            var settings = deps.leafletService.settings;
+            expect(settings.center.zoom).toBe(12);
+            expect(settings.tiles.url).toBe('http://{s}.tile.openstreetmap.fr/osmfr/{z}/{x}/{y}.png');
+            expect(settings.tiles.options.attribution).toBe('&copy; Openstreetmap France | &copy; <a href="http://www.openstreetmap.org/copyright">OpenStreetMap</a>');
+            expect(settings.tiles.options.maxZoom).toBe(19);
+        });
+        it('should handle leaflet events', function() {
+            var service = deps.leafletService;
+            var iscallbackCalled = false;
+            var isApplyed = false;
+            var callback = function (e) {
+                iscallbackCalled = true;
+            };
+            var eventRegistry = {};
+            var map = {
+                on: function (event, callback) {
+                    eventRegistry[event] = callback;
+                },
+                off: function (event, callback) {
+                    delete eventRegistry[event];
+                }
+            };
+            var scope = {'$on': function (event, callback) {
+                eventRegistry[event] = callback;
+            }, '$apply': function () {
+                isApplyed = true;
+            }};
+
+
+            expect(iscallbackCalled).toBe(false);
+            expect(isApplyed).toBe(false);
+            expect(eventRegistry.on).toBe();
+            expect(eventRegistry.off).toBe();
+
+            service.on('click', callback, map, scope);
+
+            expect(iscallbackCalled).toBe(false);
+            expect(isApplyed).toBe(false);
+            expect(eventRegistry.click).not.toBe(callback);
+            var data = {};
+            eventRegistry.click(data);
+            expect(iscallbackCalled).toBe(true);
+            expect(isApplyed).toBe(true);
+
+
+            iscallbackCalled = false;
+            isApplyed = false;
+            scope.$$phase = true; //mark that already in digest
+            eventRegistry.click(data);
+            expect(iscallbackCalled).toBe(true);
+            expect(isApplyed).toBe(false);
+        });
+    }
+
+});