Test updated: countCoverSlip moved to upper level, one more check, that handler won't being called on right click

Author: Alex Vinober

test/jasmine/tests/dragelement_test.js

@@ -30,6 +30,10 @@ describe('dragElement', function() {
 
     afterEach(destroyGraphDiv);
 
+    function countCoverSlip() {
+        return d3.selectAll('.dragcover').size();
+    }
+
     it('should init drag element', function() {
         var options = { element: this.element, gd: this.gd };
         dragElement.init(options);
@@ -103,10 +107,6 @@ describe('dragElement', function() {
     });
 
     it('should add a cover slip div to the DOM', function() {
-        function countCoverSlip() {
-            return d3.selectAll('.dragcover').size();
-        }
-
         var options = { element: this.element, gd: this.gd };
         dragElement.init(options);
 
@@ -123,15 +123,21 @@ describe('dragElement', function() {
     });
 
     it('should not add a cover slip div to the DOM when right click', function() {
-        function countCoverSlip() {
-            return d3.selectAll('.dragcover').size();
-        }
-
         var options = { element: this.element, gd: this.gd };
         dragElement.init(options);
 
+        var mockObj = {
+            handleClick: function() {}
+        };
+        spyOn(mockObj, 'handleClick');
+
+        this.element.onclick = mockObj.handleClick;
+
         mouseEvent('mousedown', this.x, this.y, { buttons: 2 });
         expect(countCoverSlip()).toEqual(0);
+
+        mouseEvent('mouseup', this.x, this.y);
+        expect(mockObj.handleClick).not.toHaveBeenCalled();
     });
 
     it('should fire off click event when down/up without dragging', function() {