Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix simulated touch event in tests #1866

Merged
merged 2 commits into from

2 participants

@fredj
Owner

No description provided.

fredj added some commits
@fredj fredj Fix simulated touch event in tests 391f81f
@fredj fredj Increase simulated clientY value
The the difference in position must be greater than
ol.pointer.MouseSource.DEDUP_DIST
9a51557
@fredj
Owner

@tsauerwein can you please take a look at this PR? thanks.

@tsauerwein
Collaborator

Thanks for fixing! Your changes are fine.

@fredj fredj merged commit 38009e1 into openlayers:master
@fredj fredj deleted the fredj:pointerevents-tests branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 17, 2014
  1. @fredj
  2. @fredj

    Increase simulated clientY value

    fredj authored
    The the difference in position must be greater than
    ol.pointer.MouseSource.DEDUP_DIST
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/spec/ol/pointer/mousesource.test.js
View
4 test/spec/ol/pointer/mousesource.test.js
@@ -40,7 +40,7 @@ describe('ol.pointer.MouseSource', function() {
// the two events are at different positions
simulateTouchEvent('touchstart', 10, 20);
- simulateEvent('mousedown', 10, 40);
+ simulateEvent('mousedown', 10, 50);
expect(eventSpy.calledTwice).to.be.ok();
});
@@ -66,7 +66,7 @@ describe('ol.pointer.MouseSource', function() {
{
identifier: 4,
clientX: x,
- clientX: y,
+ clientY: y,
target: target
}
];
Something went wrong with that request. Please try again.