From 0401a7f598ef9a36ffe1f217e1a98961046fa551 Mon Sep 17 00:00:00 2001 From: Julie Date: Mon, 15 Apr 2013 15:52:56 -0700 Subject: fix(jqLite): pass a dummy event into triggerHandler Previously, anchor elements could not be used with triggerHandler because triggerHandler passes null as the event, and any anchor element with an empty href automatically calls event.preventDefault(). Instead, pass a dummy event when using triggerHandler, similar to what full jQuery does. Modified from PR #2379. --- test/jqLiteSpec.js | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) (limited to 'test/jqLiteSpec.js') diff --git a/test/jqLiteSpec.js b/test/jqLiteSpec.js index 089ae78c..1ebe6ad4 100644 --- a/test/jqLiteSpec.js +++ b/test/jqLiteSpec.js @@ -792,7 +792,7 @@ describe('jqLite', function() { if (msie < 9){ var evnt = document.createEventObject(); evnt.srcElement = element; - evnt.relatedTarget = relatedTarget; + evnt.relatedTarget = relatedTarget; element.fireEvent('on' + type, evnt); return; }; @@ -1153,6 +1153,21 @@ describe('jqLite', function() { expect(clickSpy1).toHaveBeenCalledOnce(); expect(clickSpy2).toHaveBeenCalledOnce(); }); + + it('should pass in a dummy event', function() { + // we need the event to have at least preventDefault because angular will call it on + // all anchors with no href automatically + + var element = jqLite('poke'), + pokeSpy = jasmine.createSpy('poke'), + event; + + element.bind('click', pokeSpy); + + element.triggerHandler('click'); + event = pokeSpy.mostRecentCall.args[0]; + expect(event.preventDefault).toBeDefined(); + }); }); -- cgit v1.2.3