test(jqLite): Refactor test for isDefaultPrevent

mbenford · mgol · commit 7ffe52417154 · 2014-09-13T15:11:54.000-05:00
Refactor the spec for isDefaultPrevent method so it fails if the
existing expectations aren't executed.
diff --git a/test/jqLiteSpec.js b/test/jqLiteSpec.js
@@ -1091,15 +1091,21 @@ describe('jqLite', function() {
     });
 
     it('should have event.isDefaultPrevented method', function() {
-      jqLite(a).on('click', function(e) {
+      var element = jqLite(a),
+          clickSpy = jasmine.createSpy('clickSpy');
+
+      clickSpy.andCallFake(function(e) {
         expect(function() {
           expect(e.isDefaultPrevented()).toBe(false);
           e.preventDefault();
           expect(e.isDefaultPrevented()).toBe(true);
         }).not.toThrow();
       });
 
+      element.on('click', clickSpy);
+
       browserTrigger(a, 'click');
+      expect(clickSpy).toHaveBeenCalled();
     });
 
     it('should stop triggering handlers when stopImmediatePropagation is called', function() {
