Flush activity store state immediately on touch

functionary · Jan 12, 2016 · 04c85b5 · 04c85b5
1 parent 2a159b4
commit 04c85b5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/client/lib/stores/activity.js b/client/lib/stores/activity.js
@@ -34,7 +34,7 @@ module.exports.store = Reflux.createStore({
       lastVisit: {},
     }
     this._active = {}
-    this._flushActivityThrottled = _.throttle(this._flushActivity, this.flushTime, {leading: false})
+    this._flushActivityThrottled = _.throttle(this._flushActivity, this.flushTime)
     this._setIdleDebounced = _.debounce(this._setInactive, this.idleTime)
   },
 

diff --git a/client/test/activity-store-test.js b/client/test/activity-store-test.js
@@ -110,7 +110,11 @@ describe('activity store', () => {
     })
 
     it('should be written within ' + activity.store.flushTime + 'ms of activity', () => {
+      const firstTouchTime = Date.now()
       activity.store.touch(roomName)
+      sinon.assert.calledOnce(storage.setRoom)
+      sinon.assert.calledWithExactly(storage.setRoom, roomName, 'lastActive', firstTouchTime)
+      storage.setRoom.reset()
       clock.tick(1000)
       const lastTouchTime = Date.now()
       activity.store.touch(roomName)