Merge pull request angular-ui#1432 from nmccready/issue_1271_circle_events

Circle Specs

Author: nmccready

spec/coffee/directives/api/circle.spec.coffee

@@ -4,6 +4,7 @@ describe 'uiGmapCircle', ->
   modelClicked = false
 
   afterEach ->
+    window.google.maps.event.clearListeners()
     GCircle.resetInstances()
     modelClicked = false
 
@@ -46,6 +47,7 @@ describe 'uiGmapCircle', ->
       visible: true
       events:
         radius_changed: (gObject) ->
+        center_changed: (gObject) ->
       control: {}
 
     apiMock = window['uiGmapInitiator']
@@ -56,10 +58,8 @@ describe 'uiGmapCircle', ->
     @injects.push (uiGmapCircle) =>
       @subject = uiGmapCircle
 
-    @circle.events =
-      radius_changed: ->
-
-    _spy = spyOn @circle.events, 'radius_changed'
+    spyOn @circle.events, 'radius_changed'
+    spyOn @circle.events, 'center_changed'
 
     map.circle = @circle
 
@@ -78,7 +78,6 @@ describe 'uiGmapCircle', ->
   describe 'events', ->
     it 'call radius changed once', (done) ->
 
-
       listener = GCircle.creationSubscribe @, (gObject) =>
         _.delay =>
           window.google.maps.event.fireListener(gObject,'radius_changed')
@@ -92,6 +91,66 @@ describe 'uiGmapCircle', ->
           GCircle.creationUnSubscribe listener
         , 500
 
+    describe "updates gObject from model", ->
+      it 'change center', (done) ->
+        #issue 1271
+        @digest =>
+          @timeout =>
+            @circle.center =
+              longitude: 50
+              latitude: -50
+            # @circle.radius = 1
+            @digest =>
+              @timeout =>
+                gTestObject = @circle.control.getCircle().getCenter()
+                expect(gTestObject?.lng()).toBe(50)
+                expect(gTestObject?.lat()).toBe(-50)
+                done()
+          , 500
+
+    # describe "updates model from gObject", ->
+    #   it 'change center', (done) ->
+    #     listener = GCircle.creationSubscribe @, (gObject) =>
+    #       _.delay =>
+    #         gObject.setCenter
+    #           lng: -> 50
+    #           lat: -> -50
+    #         @digest =>
+    #           @timeout =>
+    #             expect(@circle.center.latitude).toBe(-50)
+    #             expect(@circle.center.longitude).toBe(50)
+    #             done()
+    #     @digest =>
+    #       @timeout =>
+    #         GCircle.creationUnSubscribe listener
+    #       , 500
+
+    it 'change radius does not fire center_changed', (done) ->
+      listener = GCircle.creationSubscribe @, (gObject) =>
+        _.delay =>
+          @digest =>
+            gObject.setRadius 200
+            expect(@circle.events.radius_changed).toHaveBeenCalled()
+            expect(@circle.events.center_changed).not.toHaveBeenCalled()
+            done()
+      @digest =>
+        @timeout =>
+          GCircle.creationUnSubscribe listener
+        , 500
+
+    it 'change center does not fire radius_changed', (done) ->
+      listener = GCircle.creationSubscribe @, (gObject) =>
+        _.delay =>
+          @digest =>
+            gObject.setCenter
+              lng: -> 50
+              lat: -> -50
+            expect(@circle.events.center_changed).toHaveBeenCalled()
+            expect(@circle.events.radius_changed).not.toHaveBeenCalled()
+            done()
+      @digest =>
+        GCircle.creationUnSubscribe listener
+
   it 'exists', ->
     expect(@subject).toBeDefined()
 
@@ -100,4 +159,4 @@ describe 'uiGmapCircle', ->
       @timeout =>
         expect(@circle.control.getCircle).toBeDefined()
         expect(@circle.control.getCircle()).toEqual(jasmine.any(GCircle))
-        done()
+        done()

spec/coffee/helpers/google-api-mock.coffee

@@ -140,6 +140,8 @@ angular.module('uiGmapgoogle-maps.mocks', ['uiGmapgoogle-maps'])
         @props.forEach (p) =>
           @["set#{capitalize p}"] = (val) =>
             @[p] = val
+            if p == "radius" or p == "center"
+              window.google.maps.event.fireAllListeners "#{p}_changed", @
 
       setOptions: (o)=>
         super(o)
@@ -363,10 +365,13 @@ angular.module('uiGmapgoogle-maps.mocks', ['uiGmapgoogle-maps'])
             toPush = {}
             toPush.obj = thing
             toPush.events = {}
-            toPush.events[eventName] = callBack
+            toPush.events[eventName] = [callBack]
             listeners.push toPush
           else
-            found.events[eventName] = callBack
+            if !found.events[eventName]
+              found.events[eventName] = [callBack]
+            else
+              found.events[eventName].push callBack
 
         event.addListenerOnce = (thing, eventName, callBack) ->
           callBack() #forcing immediate return for idle so async api kicks off
@@ -386,7 +391,8 @@ angular.module('uiGmapgoogle-maps.mocks', ['uiGmapgoogle-maps'])
         event.fireListener = (thing, eventName) ->
           found = _.find listeners, (obj)->
             obj.obj == thing
-          found.events[eventName](found.obj) if found? and found?.events[eventName]?
+          if found? and found?.events[eventName]?
+            found.events[eventName].forEach (cb) -> cb(found.obj)
 
       unless event.normalizedEvents
         event.normalizedEvents = ->
@@ -402,7 +408,7 @@ angular.module('uiGmapgoogle-maps.mocks', ['uiGmapgoogle-maps'])
         event.fireAllListeners = (eventName, state) ->
           listeners.forEach (obj)->
             if obj.events[eventName]?
-              obj.events[eventName](state)
+              obj.events[eventName].forEach (cb) -> cb(state)
 
       window.google.maps.event = event
       return listeners