fix: update timestamp (#60)

* fix: update timestamp

* test: new

lib/constants/flags.ts

@@ -36,8 +36,13 @@ export enum Frame {
   CALLED_COMMAND = 'calledCommand',
 }
 
+export enum Variables {
+  TIMESTAMP = 'timestamp',
+}
+
 export default {
   Storage,
   Turn,
   Frame,
+  Variables,
 };

lib/constants/index.ts

@@ -1,4 +1,4 @@
 // eslint-disable-next-line import/prefer-default-export
-export { default as Flags, Turn as T, Storage as S, Frame as F } from './flags';
+export { default as Flags, Turn as T, Storage as S, Frame as F, Variables as V } from './flags';
 
 export const TEST_VERSION_ID = '__TEST__';

lib/services/alexa/request/lifecycle/initialize.ts

@@ -1,7 +1,7 @@
 import { Context, Frame, Store } from '@voiceflow/client';
 import { HandlerInput } from 'ask-sdk';
 
-import { F, S } from '@/lib/constants';
+import { F, S, V } from '@/lib/constants';
 import { createResumeFrame, RESUME_DIAGRAM_ID } from '@/lib/services/voiceflow/diagrams/resume';
 import { StreamAction } from '@/lib/services/voiceflow/handlers/stream';
 
@@ -50,7 +50,7 @@ export const initializeGenerator = (utils: typeof utilsObj) => async (context: C
 
   // default global variables
   variables.merge({
-    timestamp: Math.floor(Date.now() / 1000),
+    [V.TIMESTAMP]: 0,
     locale: storage.get(S.LOCALE),
     user_id: storage.get(S.USER),
     sessions: storage.get(S.SESSIONS),

lib/services/alexa/request/lifecycle/update.ts

@@ -1,11 +1,12 @@
 import { Context } from '@voiceflow/client';
 
-import { T } from '@/lib/constants';
+import { T, V } from '@/lib/constants';
 
 const update = async (context: Context): Promise<void> => {
-  const { turn } = context;
+  const { turn, variables } = context;
 
   turn.set(T.REQUEST, context.getRequest());
+  variables.set(V.TIMESTAMP, Math.floor(Date.now() / 1000));
 
   await context.update();
 };

lib/services/test/index.ts

@@ -2,7 +2,7 @@ import { EventType, State } from '@voiceflow/client';
 import { StreamAction as TraceStreamAction } from '@voiceflow/client/build/lib/Context/Trace';
 import { IntentRequest as AlexaIntentRequest } from 'ask-sdk-model';
 
-import { S, T, TEST_VERSION_ID } from '@/lib/constants';
+import { S, T, TEST_VERSION_ID, V } from '@/lib/constants';
 import { StreamAction, StreamPlay } from '@/lib/services/voiceflow/handlers/stream';
 import { RequestType } from '@/lib/services/voiceflow/types';
 
@@ -29,6 +29,7 @@ const TestManager = (services: Services, config: Config, utils = utilsObj) => {
     context.setEvent(EventType.handlerWillHandle, (event) => context.trace.block(event.block.blockID));
 
     context.turn.set(T.REQUEST, request);
+    context.variables.set(V.TIMESTAMP, Math.floor(Date.now() / 1000));
 
     await context.update();
 

tests/lib/services/alexa/request/lifecycle/initialize.unit.ts

@@ -6,16 +6,6 @@ import { initializeGenerator, VAR_VF } from '@/lib/services/alexa/request/lifecy
 import { StreamAction } from '@/lib/services/voiceflow/handlers/stream';
 
 describe('initialize lifecycle unit tests', async () => {
-  let clock: sinon.SinonFakeTimers;
-
-  beforeEach(() => {
-    clock = sinon.useFakeTimers(Date.now()); // fake Date.now
-  });
-  afterEach(() => {
-    clock.restore(); // restore Date.now
-    sinon.restore();
-  });
-
   describe('initialize', () => {
     const generateFakes = () => {
       const resumeFrame = { foo: 'bar' };
@@ -129,7 +119,7 @@ describe('initialize lifecycle unit tests', async () => {
       expect(context.storage.set.args[5]).to.eql([S.REPEAT, metaObj.repeat]);
       expect(context.variables.merge.args[0]).to.eql([
         {
-          timestamp: Math.floor(clock.now / 1000),
+          timestamp: 0,
           locale,
           user_id: userId,
           sessions: 1,

tests/lib/services/alexa/request/lifecycle/update.unit.ts

@@ -1,17 +1,33 @@
 import { expect } from 'chai';
 import sinon from 'sinon';
 
-import { T } from '@/lib/constants';
+import { T, V } from '@/lib/constants';
 import update from '@/lib/services/alexa/request/lifecycle/update';
 
 describe('update lifecycle unit tests', () => {
+  let clock: sinon.SinonFakeTimers;
+
+  beforeEach(() => {
+    clock = sinon.useFakeTimers(Date.now()); // fake Date.now
+  });
+  afterEach(() => {
+    clock.restore(); // restore Date.now
+    sinon.restore();
+  });
+
   describe('update', () => {
     it('works correctly', async () => {
       const request = { foo: 'bar' };
-      const context = { turn: { set: sinon.stub() }, update: sinon.stub(), getRequest: sinon.stub().returns(request) };
+      const context = {
+        variables: { set: sinon.stub() },
+        turn: { set: sinon.stub() },
+        update: sinon.stub(),
+        getRequest: sinon.stub().returns(request),
+      };
 
       await update(context as any);
       expect(context.turn.set.args).to.eql([[T.REQUEST, request]]);
+      expect(context.variables.set.args).to.eql([[V.TIMESTAMP, Math.floor(clock.now / 1000)]]);
       expect(context.update.callCount).to.eql(1);
     });
   });

tests/lib/services/test/index.unit.ts

@@ -3,11 +3,21 @@ import { StreamAction as TraceStreamAction } from '@voiceflow/client/build/lib/C
 import { expect } from 'chai';
 import sinon from 'sinon';
 
-import { T, TEST_VERSION_ID } from '@/lib/constants';
+import { T, TEST_VERSION_ID, V } from '@/lib/constants';
 import TestManager from '@/lib/services/test';
 import { StreamAction } from '@/lib/services/voiceflow/handlers/stream';
 
 describe('test manager unit tests', () => {
+  let clock: sinon.SinonFakeTimers;
+
+  beforeEach(() => {
+    clock = sinon.useFakeTimers(Date.now()); // fake Date.now
+  });
+  afterEach(() => {
+    clock.restore(); // restore Date.now
+    sinon.restore();
+  });
+
   describe('invoke', () => {
     it('works correctly', async () => {
       const rawState = { foo: 'bar' };
@@ -25,6 +35,7 @@ describe('test manager unit tests', () => {
         stack: {
           isEmpty: sinon.stub().returns(false), // stack no empty
         },
+        variables: { set: sinon.stub() },
         update: sinon.stub(),
         getRawState: sinon.stub().returns(rawState),
         trace: { get: sinon.stub().returns(trace), block: sinon.stub() },
@@ -65,6 +76,7 @@ describe('test manager unit tests', () => {
       fn(event);
       expect(context.trace.block.args).to.eql([[event.block.blockID]]);
       expect(context.turn.set.args).to.eql([[T.REQUEST, request]]);
+      expect(context.variables.set.args).to.eql([[V.TIMESTAMP, Math.floor(clock.now / 1000)]]);
       expect(context.update.callCount).to.eql(1);
     });
 
@@ -83,6 +95,7 @@ describe('test manager unit tests', () => {
         stack: {
           isEmpty: sinon.stub().returns(true), // stack is empty
         },
+        variables: { set: sinon.stub() },
         update: sinon.stub(),
         getRawState: sinon.stub().returns(rawState),
         trace: { get: sinon.stub().returns(trace), block: sinon.stub(), end: sinon.stub() },
@@ -125,6 +138,7 @@ describe('test manager unit tests', () => {
           stack: {
             isEmpty: sinon.stub().returns(true), // stack is empty
           },
+          variables: { set: sinon.stub() },
           update: sinon.stub(),
           getRawState: sinon.stub().returns(rawState),
           trace: { get: sinon.stub().returns(trace), block: sinon.stub(), end: sinon.stub(), stream: sinon.stub() },
@@ -167,6 +181,7 @@ describe('test manager unit tests', () => {
           stack: {
             isEmpty: sinon.stub().returns(true), // stack is empty
           },
+          variables: { set: sinon.stub() },
           update: sinon.stub(),
           getRawState: sinon.stub().returns(rawState),
           trace: { get: sinon.stub().returns(trace), block: sinon.stub(), end: sinon.stub(), stream: sinon.stub() },
@@ -209,6 +224,7 @@ describe('test manager unit tests', () => {
           stack: {
             isEmpty: sinon.stub().returns(true), // stack is empty
           },
+          variables: { set: sinon.stub() },
           update: sinon.stub(),
           getRawState: sinon.stub().returns(rawState),
           trace: { get: sinon.stub().returns(trace), block: sinon.stub(), end: sinon.stub(), stream: sinon.stub() },