chore(src/events) add tap filter (#103)

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@userdive/agent",
   "description": "webpage tracking agent",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "author": "UNCOVER TRUTH Inc.",
   "bugs": {
     "url": "https://github.com/userdive/agent.js/issues"

src/events/touch.js

@@ -3,17 +3,49 @@ import EventBase from '../events'
 import { validate } from '../browser'
 import { TOUCH } from '../constants'
 
+function getFirstTouch (e: TouchEvent): Touch {
+  return e.changedTouches ? e.changedTouches[0] : e.touches[0]
+}
+
 export default class TouchEvents extends EventBase {
+  start: Touch
   validate (): boolean {
     return validate(TOUCH)
   }
-  dispatch (e: TouchEvent) {
-    const t = e.changedTouches ? e.changedTouches[0] : e.touches[0]
-    this.emit({ x: t.pageX, y: t.pageY })
-  }
   on () {
-    super.on('touchstart', this.dispatch.bind(this), 'l')
-    super.on('touchmove', this.dispatch.bind(this), 'l')
-    super.on('touchend', this.dispatch.bind(this), 'a')
+    super.on(
+      'touchstart',
+      (e: TouchEvent) => {
+        this.start = getFirstTouch(e)
+        this.emit({ x: this.start.pageX, y: this.start.pageY })
+      },
+      'l'
+    )
+
+    super.on(
+      'touchmove',
+      (e: TouchEvent) => {
+        const t = getFirstTouch(e)
+        this.emit({ x: t.pageX, y: t.pageY })
+      },
+      'l'
+    )
+
+    super.on(
+      'touchend',
+      (e: TouchEvent) => {
+        const t = getFirstTouch(e)
+        if (!this.start) {
+          return this.warning(`start is not defined`)
+        }
+        if (
+          Math.abs(this.start.pageX - t.pageX) < 10 &&
+          Math.abs(this.start.pageY - t.pageY) < 10
+        ) {
+          this.emit({ x: t.pageX, y: t.pageY })
+        }
+      },
+      'a'
+    )
   }
 }

test/events.touch.test.js

@@ -1,7 +1,6 @@
 /* @flow */
 import { describe, it, beforeEach } from 'mocha'
 import { random } from 'faker'
-import { spy as sinonSpy } from 'sinon'
 import assert from 'assert'
 
 import EventEmitter from 'events'
@@ -28,14 +27,4 @@ describeExcludeTouch()('touch', () => {
   it('validate', () => {
     assert(instance.validate())
   })
-
-  it('dispatch', () => {
-    const spy = sinonSpy(instance, 'emit')
-    const e: any = {
-      touches: [{ pageX: random.number(), pageY: random.number() }]
-    }
-    assert(instance.dispatch(e) === undefined)
-    assert(spy.called)
-    spy.restore()
-  })
 })