fix: add IMethods interface for use in time module

Author: mannyistyping

CHANGELOG.md

@@ -9,7 +9,10 @@ Please see [CONTRIBUTING.md](https://github.com/cucumber/cucumber/blob/master/CO
 
 ## [Unreleased]
 ### Added
-- `IPublishConfig` interface for use in return type of `makePublishConfig` instead of explicit `any` ([#1648](https://github.com/cucumber/cucumber-js/pull/2106))
+- `IMethods` interface for use in `getTimestamp`, `durationBetweenTimestamps`, and `wrapPromiseWithTimeout` functions and `methods` in `time` module instead of explicit `any` ([#2111](https://github.com/cucumber/cucumber-js/pull/2111))
+
+### Added
+- `IPublishConfig` interface for use in return type of `makePublishConfig` instead of explicit `any` ([#2106](https://github.com/cucumber/cucumber-js/pull/2106))
 
 ## [8.5.1] - 2022-07-28
 ### Fixed

src/time.ts

@@ -1,9 +1,16 @@
 import { performance } from 'perf_hooks'
 import * as messages from '@cucumber/messages'
+import { FakeClock, GlobalTimers, TimerId } from '@sinonjs/fake-timers'
 
 let previousTimestamp: number
 
-const methods: any = {
+interface IMethods extends GlobalTimers<TimerId> {
+  beginTiming: () => void
+  endTiming: () => number
+  performance: FakeClock<TimerId>['performance']
+}
+
+const methods: Partial<IMethods> = {
   beginTiming() {
     previousTimestamp = getTimestamp()
   },
@@ -44,7 +51,7 @@ export async function wrapPromiseWithTimeout<T>(
   timeoutInMilliseconds: number,
   timeoutMessage: string = ''
 ): Promise<T> {
-  let timeoutId: NodeJS.Timeout
+  let timeoutId: TimerId
   if (timeoutMessage === '') {
     timeoutMessage = `Action did not complete within ${timeoutInMilliseconds} milliseconds`
   }