fix(skipTouch): continues the flow with the callback when skipping touch

Author: willgm

src/express-session-etcd3.ts

@@ -119,6 +119,7 @@ export default class Etcd3Store extends Store {
   touch = (sid: string, session: Express.SessionData, callback: (err: any) => void): void => {
     if (this.config.skipTouch) {
       this.debug('SKIP TOUCH "%s"', sid)
+      callback(null)
       return
     }
     this.debug('TOUCH "%s" %O', sid, session)

test/express-session-etcd3.test.ts

@@ -182,12 +182,13 @@ describe('Etcd3Store test suit', () => {
       expect(subject.set).toHaveBeenCalledWith(sessionData.sid, sessionData, callback)
     })
 
-    it('should not set it again if skipTouch is true at the adapter config', async () => {
+    it('should return the callback without set it again if skipTouch is true at the adapter config', async () => {
       const { subject, client } = await createSubject({ skipTouch: true })
       jest.spyOn(subject, 'set').mockImplementation(jest.fn())
       const callback = jest.fn()
       subject.touch(sessionData.sid, sessionData, callback)
       expect(subject.set).not.toHaveBeenCalled()
+      expect(callback).toBeCalledWith(null)
     })
   })