Merge pull request #1658 from hypothesis/fix-adder-event-handler

Fix adder event handler in Annotator.Guest

h/static/scripts/guest.coffee

@@ -350,7 +350,7 @@ class Annotator.Guest extends Annotator
       method: 'addToken'
       params: token
 
-  onAdderMouseup: ->
+  onAdderMouseup: (event) ->
     event.preventDefault()
     event.stopPropagation()
 

karma.config.js

@@ -46,6 +46,7 @@ module.exports = function(config) {
       'h/static/scripts/auth.js',
       'h/static/scripts/helpers.js',
       'h/static/scripts/session.js',
+      'h/static/scripts/hypothesis.js',
       'h/static/scripts/vendor/sinon.js',
       'h/static/scripts/vendor/chai.js',
       'h/templates/*.html',

tests/js/guest-test.coffee

+assert = chai.assert
+sinon.assert.expose(assert, prefix: '')
+
+describe 'Annotator.Guest', ->
+  createGuest = (options) ->
+    element = document.createElement('div')
+    return new Annotator.Guest(element, options || {})
+
+  # Silence Annotator's sassy backchat
+  before -> sinon.stub(console, 'log')
+  after -> console.log.restore()
+
+  describe 'onAdderMouseUp', ->
+    it 'it prevents the default browser action when triggered', () ->
+      event = jQuery.Event('mouseup')
+      guest = createGuest()
+      guest.onAdderMouseup(event)
+      assert.isTrue(event.isDefaultPrevented())
+
+    it 'it stops any further event bubbling', () ->
+      event = jQuery.Event('mouseup')
+      guest = createGuest()
+      guest.onAdderMouseup(event)
+      assert.isTrue(event.isPropagationStopped())