fix(execution): add started and done timestamps (#237)

Author: tomiiide

src/core/StatusManager.ts

@@ -40,12 +40,14 @@ export class StatusManager {
       step.execution = {
         status: 'PENDING',
         process: [],
+        startedAt: Date.now(),
       }
       this.updateStepInRoute(step)
     }
 
     // Change status to PENDING after resuming from FAILED
     if (step.execution.status === 'FAILED') {
+      step.execution.startedAt = Date.now()
       step.execution.status = 'PENDING'
       this.updateStepInRoute(step)
     }
@@ -69,6 +71,9 @@ export class StatusManager {
       throw Error("Can't update empty execution.")
     }
     step.execution.status = status
+    if (status === 'DONE') {
+      step.execution.doneAt = Date.now()
+    }
     if (execution) {
       step.execution = {
         ...step.execution,
@@ -162,9 +167,6 @@ export class StatusManager {
     }
 
     switch (status) {
-      case 'STARTED':
-        currentProcess.startedAt = Date.now()
-        break
       case 'CANCELLED':
         currentProcess.doneAt = Date.now()
         break

src/core/StatusManager.unit.spec.ts

@@ -1,7 +1,11 @@
 import type { Route } from '@lifi/types'
 import type { Mock } from 'vitest'
 import { beforeAll, beforeEach, describe, expect, it, vi } from 'vitest'
-import { buildRouteObject, buildStepObject } from '../../tests/fixtures.js'
+import {
+  SOME_DATE,
+  buildRouteObject,
+  buildStepObject,
+} from '../../tests/fixtures.js'
 import { setupTestEnvironment } from '../../tests/setup.js'
 import { StatusManager } from './StatusManager.js'
 import { executionState } from './executionState.js'
@@ -45,6 +49,7 @@ describe('StatusManager', () => {
 
   beforeEach(() => {
     updateRouteHookMock = vi.fn()
+    vi.spyOn(Date, 'now').mockImplementation(() => SOME_DATE)
   })
 
   describe('initExecutionObject', () => {
@@ -59,6 +64,7 @@ describe('StatusManager', () => {
           execution: {
             status: 'PENDING',
             process: [],
+            startedAt: SOME_DATE,
           },
         })
 
@@ -83,6 +89,9 @@ describe('StatusManager', () => {
   })
 
   describe('updateExecution', () => {
+    beforeEach(() => {
+      vi.spyOn(Date, 'now').mockImplementation(() => SOME_DATE + 10)
+    })
     describe('when no execution is defined yet', () => {
       beforeEach(() => {
         statusManager = initializeStatusManager({ includingExecution: false })

src/core/types.ts

@@ -162,6 +162,8 @@ export type Process = {
 }
 
 export interface Execution {
+  startedAt: number
+  doneAt?: number
   status: ExecutionStatus
   process: Array<Process>
   fromAmount?: string

tests/fixtures.ts

@@ -14,7 +14,7 @@ const SOME_OTHER_TOKEN: Token = {
   priceUSD: '',
 }
 
-const SOME_DATE = new Date('2021-04-10').getTime()
+export const SOME_DATE = new Date('2021-04-10').getTime()
 
 export const buildStepObject = ({
   includingExecution = true,
@@ -124,6 +124,8 @@ export const buildStepObject = ({
   execution: includingExecution
     ? {
         status: 'PENDING',
+        startedAt: SOME_DATE,
+        doneAt: SOME_DATE + 10,
         process: [
           {
             type: 'TOKEN_ALLOWANCE',