@fluidframework/test-utils 2.74.0-365691 → 2.74.0

package/src/loaderContainerTracker.ts

@@ -61,7 +61,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * Add a loader to start to track any container created from them
 	 * @param loader - loader to start tracking any container created.
 	 */
-	public add<LoaderType extends IHostLoader>(loader: LoaderType) {
+	public add<LoaderType extends IHostLoader>(loader: LoaderType): void {
 		// TODO: Expose Loader API to able to intercept container creation (See issue #5114)
 		const patch = <T, C extends IContainer>(fn: (...args) => Promise<C>) => {
 			const boundFn = fn.bind(loader);
@@ -85,7 +85,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 *
 	 * @param container - container to add
 	 */
-	public addContainer(container: IContainer) {
+	public addContainer(container: IContainer): void {
 		// don't add container that is already tracked
 		if (this.containers.has(container)) {
 			return;
@@ -144,7 +144,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * @param container - the container to track
 	 * @param record - the record to update the trailing op information
 	 */
-	private trackTrailingNoOps(container: IContainer, record: ContainerRecord) {
+	private trackTrailingNoOps(container: IContainer, record: ContainerRecord): void {
 		const deltaManagerFull = toIDeltaManagerFull(container.deltaManager);
 		deltaManagerFull.outbound.on("op", (messages) => {
 			for (const msg of messages) {
@@ -184,7 +184,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 		});
 	}
 
-	private trackLastProposal(container: IContainer) {
+	private trackLastProposal(container: IContainer): void {
 		container.on("codeDetailsProposed", (value, proposal) => {
 			if (proposal.sequenceNumber > this.lastProposalSeqNum) {
 				this.lastProposalSeqNum = proposal.sequenceNumber;
@@ -195,7 +195,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	/**
 	 * Reset the tracker, closing all containers and stop tracking them.
 	 */
-	public reset() {
+	public reset(): void {
 		this.lastProposalSeqNum = 0;
 		for (const container of this.containers.keys()) {
 			container.close();
@@ -305,7 +305,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 *
 	 * @param containersToApply - the set of containers to check
 	 */
-	private getPendingClients(containersToApply: IContainer[]) {
+	private getPendingClients(containersToApply: IContainer[]): [IContainer, Set<string>][] {
 		// All the clientId we track should be a superset of the quorum, otherwise, we are missing
 		// leave messages
 		const openedDocuments = Array.from(this.containers.keys()).filter((c) => !c.closed);
@@ -338,7 +338,9 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 *
 	 * @param containersToApply - the set of containers to check
 	 */
-	private needSequenceNumberSynchronize(containersToApply: NonEmptyArray<IContainer>) {
+	private needSequenceNumberSynchronize(
+		containersToApply: NonEmptyArray<IContainer>,
+	): { reason: string; message: string } | undefined {
 		// If there is a pending proposal, wait for it to be accepted
 		const minSeqNum = containersToApply[0].deltaManager.minimumSequenceNumber;
 		if (minSeqNum < this.lastProposalSeqNum) {
@@ -405,7 +407,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 		return undefined;
 	}
 
-	private containerIndexStrings(containers: IContainer[]) {
+	private containerIndexStrings(containers: IContainer[]): number[] {
 		return containers.map(
 			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
 			(c) => this.containers.get(c)!.index,
@@ -420,18 +422,20 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 *
 	 * @param containersToApply - the set of containers to wait for any inbound ops for
 	 */
-	private async waitForPendingClients(pendingClients: [IContainer, Set<string>][]) {
+	private async waitForPendingClients(
+		pendingClients: [IContainer, Set<string>][],
+	): Promise<void[]> {
 		const unconnectedClients = Array.from(this.containers.keys()).filter(
 			(c) => !c.closed && c.connectionState !== ConnectionState.Connected,
 		);
 		return Promise.all(
 			pendingClients.map(async ([container, pendingClientId]) => {
 				return new Promise<void>((resolve) => {
-					const cleanup = () => {
+					const cleanup = (): void => {
 						unconnectedClients.forEach((c) => c.off("connected", handler));
 						container.getQuorum().off("removeMember", handler);
 					};
-					const handler = (clientId: string) => {
+					const handler = (clientId: string): void => {
 						pendingClientId.delete(clientId);
 						if (pendingClientId.size === 0) {
 							cleanup();
@@ -458,9 +462,9 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * Utility to wait for any inbound ops from a set of containers
 	 * @param containersToApply - the set of containers to wait for any inbound ops for
 	 */
-	private async waitForAnyInboundOps(containersToApply: IContainer[]) {
+	private async waitForAnyInboundOps(containersToApply: IContainer[]): Promise<void> {
 		return new Promise<void>((resolve) => {
-			const handler = () => {
+			const handler = (): void => {
 				containersToApply.map((c) => {
 					toIDeltaManagerFull(c.deltaManager).inbound.off("push", handler);
 				});
@@ -475,7 +479,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	/**
 	 * Resume all queue activities on all paused tracked containers and return them
 	 */
-	public resumeProcessing(...containers: IContainer[]) {
+	public resumeProcessing(...containers: IContainer[]): IContainer[] {
 		const resumed: IContainer[] = [];
 		const containersToApply = this.getContainers(containers);
 		for (const container of containersToApply) {
@@ -505,7 +509,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * avoid missing join messages or change the sequence of event when switching from read to
 	 * write mode.
 	 */
-	public async pauseProcessing(...containers: IContainer[]) {
+	public async pauseProcessing(...containers: IContainer[]): Promise<void> {
 		const waitP: Promise<void>[] = [];
 		const containersToApply = this.getContainers(containers);
 		for (const container of containersToApply) {
@@ -529,7 +533,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * @param container - the container to pause
 	 * @param record - the record for the container
 	 */
-	private async pauseContainer(container: IContainer, record: ContainerRecord) {
+	private async pauseContainer(container: IContainer, record: ContainerRecord): Promise<void> {
 		debugWait(`${record.index}: pausing container`);
 		const deltaManagerFull = toIDeltaManagerFull(container.deltaManager);
 		assert(!deltaManagerFull.outbound.paused, "Container should not be paused yet");
@@ -593,7 +597,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 *
 	 * Pausing will switch the container to write mode. See `pauseProcessing`
 	 */
-	public async processIncoming(...containers: IContainer[]) {
+	public async processIncoming(...containers: IContainer[]): Promise<void> {
 		return this.processQueue(
 			containers,
 			(container) => toIDeltaManagerFull(container.deltaManager).inbound,
@@ -607,7 +611,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 *
 	 * Pausing will switch the container to write mode. See `pauseProcessing`
 	 */
-	public async processOutgoing(...containers: IContainer[]) {
+	public async processOutgoing(...containers: IContainer[]): Promise<void> {
 		return this.processQueue(
 			containers,
 			(container) => toIDeltaManagerFull(container.deltaManager).outbound,
@@ -620,7 +624,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	private async processQueue<U>(
 		containers: IContainer[],
 		getQueue: (container: IContainer) => IDeltaQueue<U>,
-	) {
+	): Promise<void> {
 		await this.pauseProcessing(...containers);
 		const resumed: IDeltaQueue<U>[] = [];
 
@@ -672,15 +676,18 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * @param container - the container to setup
 	 * @param inflightTracker - a map to track the clientSequenceNumber per container it expect to get ops back
 	 */
-	private setupInOutTracker(container: IContainer, inflightTracker: Map<IContainer, number>) {
-		const outHandler = (messages: IDocumentMessage[]) => {
+	private setupInOutTracker(
+		container: IContainer,
+		inflightTracker: Map<IContainer, number>,
+	): () => void {
+		const outHandler = (messages: IDocumentMessage[]): void => {
 			for (const message of messages) {
 				if (!canBeCoalescedByService(message)) {
 					inflightTracker.set(container, message.clientSequenceNumber);
 				}
 			}
 		};
-		const inHandler = (message: ISequencedDocumentMessage) => {
+		const inHandler = (message: ISequencedDocumentMessage): void => {
 			if (
 				!canBeCoalescedByService(message) &&
 				message.clientId === container.clientId &&
@@ -694,7 +701,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 		deltaManagerFull.outbound.on("op", outHandler);
 		deltaManagerFull.inbound.on("push", inHandler);
 
-		return () => {
+		return (): void => {
 			deltaManagerFull.outbound.off("op", outHandler);
 			deltaManagerFull.inbound.off("push", inHandler);
 		};
@@ -703,9 +710,9 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	/**
 	 * Setup debug traces for connection and ops
 	 */
-	private setupTrace(container: IContainer, index: number) {
+	private setupTrace(container: IContainer, index: number): void {
 		if (debugOp.enabled) {
-			const getContentsString = (type: string, msgContents: any) => {
+			const getContentsString = (type: string, msgContents: any): string => {
 				try {
 					if (type !== MessageType.Operation) {
 						if (typeof msgContents === "string") {
@@ -779,7 +786,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
 	 * @param containers - The container to filter to.  If the array is empty, it means don't filter and return
 	 * all open containers.
 	 */
-	private getContainers(containers: IContainer[]) {
+	private getContainers(containers: IContainer[]): IContainer[] {
 		const containersToApply =
 			containers.length === 0 ? Array.from(this.containers.keys()) : containers;
 		return containersToApply.filter((container) => !container.closed);

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/test-utils";
-export const pkgVersion = "2.74.0-365691";
+export const pkgVersion = "2.74.0";

package/src/testContainerRuntimeFactory.ts

@@ -68,6 +68,7 @@ interface backCompat_ContainerRuntime {
  */
 export const createTestContainerRuntimeFactory = (
 	containerRuntimeCtor: typeof ContainerRuntime,
+	// eslint-disable-next-line @typescript-eslint/explicit-function-return-type -- Returning anonymous class type
 ) => {
 	return class extends RuntimeFactoryHelper {
 		constructor(
@@ -144,14 +145,17 @@ export const createTestContainerRuntimeFactory = (
 					existing,
 				);
 			}
-			const provideEntryPoint = async (runtime: IContainerRuntime) => {
+			const provideEntryPoint = async (runtime: IContainerRuntime): Promise<FluidObject> => {
 				const entryPoint = await runtime.getAliasedDataStoreEntryPoint("default");
 				if (entryPoint === undefined) {
 					throw new Error("default dataStore must exist");
 				}
 				return entryPoint.get();
 			};
-			const getDefaultObject = async (request: IRequest, runtime: IContainerRuntime) => {
+			const getDefaultObject = async (
+				request: IRequest,
+				runtime: IContainerRuntime,
+			): Promise<IResponse | undefined> => {
 				const parser = RequestParser.create(request);
 				if (parser.pathParts.length === 0) {
 					// This cast is safe as loadContainerRuntime is called below

package/src/testContainerRuntimeFactoryWithDefaultDataStore.ts

@@ -14,7 +14,7 @@ import {
 	NamedFluidDataStoreRegistryEntries,
 } from "@fluidframework/runtime-definitions/internal";
 
-const getDefaultFluidObject = async (runtime: IContainerRuntime) => {
+const getDefaultFluidObject = async (runtime: IContainerRuntime): Promise<FluidObject> => {
 	const entryPoint = await runtime.getAliasedDataStoreEntryPoint("default");
 	if (entryPoint === undefined) {
 		throw new Error("default dataStore must exist");

package/src/testFluidObject.ts

@@ -41,11 +41,11 @@ import { ITestFluidObject } from "./interfaces.js";
  * @internal
  */
 export class TestFluidObject implements ITestFluidObject {
-	public get ITestFluidObject() {
+	public get ITestFluidObject(): ITestFluidObject {
 		return this;
 	}
 
-	public get IFluidLoadable() {
+	public get IFluidLoadable(): IFluidLoadable {
 		return this;
 	}
 
@@ -101,8 +101,8 @@ export class TestFluidObject implements ITestFluidObject {
 			: create404Response(request);
 	}
 
-	public async initialize(existing: boolean) {
-		const doInitialization = async () => {
+	public async initialize(existing: boolean): Promise<void> {
+		const doInitialization = async (): Promise<void> => {
 			if (!existing) {
 				this.root = SharedMap.create(this.runtime, "root");
 
@@ -176,7 +176,7 @@ export type TestDataObjectKind = new (
  * @internal
  */
 export class TestFluidObjectFactory implements IFluidDataStoreFactory {
-	public get IFluidDataStoreFactory() {
+	public get IFluidDataStoreFactory(): IFluidDataStoreFactory {
 		return this;
 	}
 

package/src/testFluidObjectInternal.ts

@@ -35,7 +35,7 @@ import type {
  * @internal
  */
 export class TestFluidObjectInternal implements IFluidLoadable {
-	public get IFluidLoadable() {
+	public get IFluidLoadable(): IFluidLoadable {
 		return this;
 	}
 
@@ -94,8 +94,8 @@ export class TestFluidObjectInternal implements IFluidLoadable {
 			: create404Response(request);
 	}
 
-	public async initialize(existing: boolean) {
-		const doInitialization = async () => {
+	public async initialize(existing: boolean): Promise<void> {
+		const doInitialization = async (): Promise<void> => {
 			if (!existing) {
 				for (const [key, sharedObjectFactory] of this.initialSharedObjectsFactories) {
 					const channel = this.runtime.createChannel(key, sharedObjectFactory.type);

package/src/testObjectProvider.ts

@@ -365,7 +365,7 @@ export class EventAndErrorTrackingLogger
 	private readonly expectedEvents: { index: number; event: ITelemetryGenericEventExt }[] = [];
 	private readonly unexpectedErrors: ITelemetryBaseEvent[] = [];
 
-	public registerExpectedEvent(...orderedExpectedEvents: ITelemetryGenericEventExt[]) {
+	public registerExpectedEvent(...orderedExpectedEvents: ITelemetryGenericEventExt[]): void {
 		if (this.expectedEvents.length !== 0) {
 			// we don't have to error here. just no reason not to. given the events must be
 			// ordered it could be tricky to figure out problems around multiple registrations.
@@ -419,7 +419,10 @@ export class EventAndErrorTrackingLogger
 		this.baseLogger?.send(event);
 	}
 
-	public reportAndClearTrackedEvents() {
+	public reportAndClearTrackedEvents(): {
+		expectedNotFound: { index: number; event: ITelemetryGenericEventExt }[];
+		unexpectedErrors: ITelemetryBaseEvent[];
+	} {
 		const expectedNotFound = this.expectedEvents.splice(0, this.expectedEvents.length);
 		const unexpectedErrors = this.unexpectedErrors.splice(0, this.unexpectedErrors.length);
 		return {
@@ -490,7 +493,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 		return this._logger;
 	}
 
-	public get tracker() {
+	public get tracker(): EventAndErrorTrackingLogger {
 		void this.logger;
 		assert(this._tracker !== undefined, "should be initialized");
 		return this._tracker;
@@ -499,7 +502,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.documentServiceFactory}
 	 */
-	public get documentServiceFactory() {
+	public get documentServiceFactory(): IDocumentServiceFactory {
 		if (!this._documentServiceFactory) {
 			this._documentServiceFactory = this.driver.createDocumentServiceFactory();
 		}
@@ -509,7 +512,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.urlResolver}
 	 */
-	public get urlResolver() {
+	public get urlResolver(): IUrlResolver {
 		if (!this._urlResolver) {
 			this._urlResolver = this.driver.createUrlResolver();
 		}
@@ -519,14 +522,14 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.documentId}
 	 */
-	public get documentId() {
+	public get documentId(): string {
 		return this._documentIdStrategy.get();
 	}
 
 	/**
 	 * {@inheritDoc ITestObjectProvider.defaultCodeDetails}
 	 */
-	public get defaultCodeDetails() {
+	public get defaultCodeDetails(): IFluidCodeDetails {
 		return defaultCodeDetails;
 	}
 
@@ -543,7 +546,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	public createLoader(
 		packageEntries: Iterable<[IFluidCodeDetails, fluidEntryPoint]>,
 		loaderProps?: Partial<ILoaderProps>,
-	) {
+	): Loader {
 		const logger = createMultiSinkLogger({
 			loggers: [this.logger, loaderProps?.logger],
 		});
@@ -566,7 +569,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	public async createContainer(
 		entryPoint: fluidEntryPoint,
 		loaderProps?: Partial<ILoaderProps>,
-	) {
+	): Promise<IContainer> {
 		if (this._documentCreated) {
 			throw new Error(
 				"Only one container/document can be created. To load the container/document use loadContainer",
@@ -632,7 +635,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 		loader: ILoader,
 		headers?: IRequestHeader,
 		pendingLocalState?: string,
-	) {
+	): Promise<IContainer> {
 		return loader.resolve(
 			{
 				url: await this.driver.createContainerUrl(this.documentId),
@@ -645,7 +648,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.makeTestLoader}
 	 */
-	public makeTestLoader(testContainerConfig?: ITestContainerConfig) {
+	public makeTestLoader(testContainerConfig?: ITestContainerConfig): Loader {
 		return this.createLoader(
 			[[defaultCodeDetails, this.createFluidEntryPoint(testContainerConfig)]],
 			testContainerConfig?.loaderProps,
@@ -695,7 +698,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.reset}
 	 */
-	public reset() {
+	public reset(): void {
 		this._loaderContainerTracker.reset();
 		this._documentServiceFactory = undefined;
 		this._urlResolver = undefined;
@@ -716,7 +719,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 		return this._loaderContainerTracker.ensureSynchronized(...containers);
 	}
 
-	private async waitContainerToCatchUp(container: IContainer) {
+	private async waitContainerToCatchUp(container: IContainer): Promise<boolean> {
 		// The original waitContainerToCatchUp() from container loader uses either Container.resume()
 		// or Container.connect() as part of its implementation. However, resume() was deprecated
 		// and eventually replaced with connect(). To avoid issues during LTS compatibility testing
@@ -731,7 +734,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.updateDocumentId}
 	 */
-	public updateDocumentId(resolvedUrl: IResolvedUrl | undefined) {
+	public updateDocumentId(resolvedUrl: IResolvedUrl | undefined): void {
 		this._documentIdStrategy.update(resolvedUrl);
 		this.logger.send({
 			category: "generic",
@@ -743,7 +746,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	/**
 	 * {@inheritDoc ITestObjectProvider.resetLoaderContainerTracker}
 	 */
-	public resetLoaderContainerTracker(syncSummarizerClients: boolean = false) {
+	public resetLoaderContainerTracker(syncSummarizerClients: boolean = false): void {
 		this._loaderContainerTracker.reset();
 		this._loaderContainerTracker = new LoaderContainerTracker(syncSummarizerClients);
 	}
@@ -795,7 +798,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.logger}
 	 */
-	public get logger() {
+	public get logger(): ITelemetryBaseLogger {
 		if (this._logger === undefined) {
 			this._tracker = new EventAndErrorTrackingLogger(getTestLogger?.());
 			this._logger = createChildLogger({
@@ -806,7 +809,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 		return this._logger;
 	}
 
-	public get tracker() {
+	public get tracker(): EventAndErrorTrackingLogger {
 		void this.logger;
 		assert(this._tracker !== undefined, "should be initialized");
 		return this._tracker;
@@ -815,7 +818,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.documentServiceFactory}
 	 */
-	public get documentServiceFactory() {
+	public get documentServiceFactory(): IDocumentServiceFactory {
 		if (!this._documentServiceFactory) {
 			this._documentServiceFactory = this.driverForCreating.createDocumentServiceFactory();
 		}
@@ -825,7 +828,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.urlResolver}
 	 */
-	public get urlResolver() {
+	public get urlResolver(): IUrlResolver {
 		if (!this._urlResolver) {
 			this._urlResolver = this.driverForCreating.createUrlResolver();
 		}
@@ -835,14 +838,14 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.documentId}
 	 */
-	public get documentId() {
+	public get documentId(): string {
 		return this._documentIdStrategy.get();
 	}
 
 	/**
 	 * {@inheritDoc ITestObjectProvider.defaultCodeDetails}
 	 */
-	public get defaultCodeDetails() {
+	public get defaultCodeDetails(): IFluidCodeDetails {
 		return defaultCodeDetails;
 	}
 
@@ -874,7 +877,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	private createLoaderForCreating(
 		packageEntries: Iterable<[IFluidCodeDetails, fluidEntryPoint]>,
 		loaderProps?: Partial<ILoaderProps>,
-	) {
+	): Loader {
 		const logger = createMultiSinkLogger({
 			loggers: [this.logger, loaderProps?.logger],
 		});
@@ -895,7 +898,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	private createLoaderForLoading(
 		packageEntries: Iterable<[IFluidCodeDetails, fluidEntryPoint]>,
 		loaderProps?: Partial<ILoaderProps>,
-	) {
+	): Loader {
 		const logger = createMultiSinkLogger({
 			loggers: [this.logger, loaderProps?.logger],
 		});
@@ -920,7 +923,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 		packageEntries: Iterable<[IFluidCodeDetails, fluidEntryPoint]>,
 		loaderProps?: Partial<ILoaderProps>,
 		forceUseCreateVersion = false,
-	) {
+	): Loader {
 		const useCreateVersion = forceUseCreateVersion === true || this.useCreateApi;
 		if (this.useCreateApi) {
 			// After we create the first loader, we can set this.useCreateApi to false.
@@ -938,7 +941,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	public async createContainer(
 		entryPoint: fluidEntryPoint,
 		loaderProps?: Partial<ILoaderProps>,
-	) {
+	): Promise<IContainer> {
 		if (this._documentCreated) {
 			throw new Error(
 				"Only one container/document can be created. To load the container/document use loadContainer",
@@ -1006,7 +1009,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 		headers?: IRequestHeader,
 		driver?: ITestDriver,
 		pendingLocalState?: string,
-	) {
+	): Promise<IContainer> {
 		return loader.resolve(
 			{
 				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
@@ -1020,7 +1023,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.makeTestLoader}
 	 */
-	public makeTestLoader(testContainerConfig?: ITestContainerConfig) {
+	public makeTestLoader(testContainerConfig?: ITestContainerConfig): Loader {
 		return this.createLoader(
 			[[defaultCodeDetails, this.createFluidEntryPoint(testContainerConfig)]],
 			testContainerConfig?.loaderProps,
@@ -1080,7 +1083,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.reset}
 	 */
-	public reset() {
+	public reset(): void {
 		this.useCreateApi = true;
 		this._loaderContainerTracker.reset();
 		this._logger = undefined;
@@ -1102,7 +1105,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 		return this._loaderContainerTracker.ensureSynchronized(...containers);
 	}
 
-	private async waitContainerToCatchUp(container: IContainer) {
+	private async waitContainerToCatchUp(container: IContainer): Promise<boolean> {
 		// The original waitContainerToCatchUp() from container loader uses either Container.resume()
 		// or Container.connect() as part of its implementation. However, resume() was deprecated
 		// and eventually replaced with connect(). To avoid issues during LTS compatibility testing
@@ -1117,7 +1120,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.updateDocumentId}
 	 */
-	public updateDocumentId(resolvedUrl: IResolvedUrl | undefined) {
+	public updateDocumentId(resolvedUrl: IResolvedUrl | undefined): void {
 		this._documentIdStrategy.update(resolvedUrl);
 		this.logger.send({
 			category: "generic",
@@ -1129,7 +1132,7 @@ export class TestObjectProviderWithVersionedLoad implements ITestObjectProvider
 	/**
 	 * {@inheritDoc ITestObjectProvider.resetLoaderContainerTracker}
 	 */
-	public resetLoaderContainerTracker(syncSummarizerClients: boolean = false) {
+	public resetLoaderContainerTracker(syncSummarizerClients: boolean = false): void {
 		this._loaderContainerTracker.reset();
 		this._loaderContainerTracker = new LoaderContainerTracker(syncSummarizerClients);
 	}
@@ -1169,7 +1172,7 @@ const primaryEventProps = ({
 	eventName,
 	error,
 	errorType,
-}: ITelemetryBaseEvent) => ({
+}: ITelemetryBaseEvent): Partial<ITelemetryBaseEvent> => ({
 	category,
 	eventName,
 	error,
@@ -1183,7 +1186,7 @@ const primaryEventProps = ({
 export function getUnexpectedLogErrorException(
 	logger: IEventAndErrorTrackingLogger | undefined,
 	prefix?: string,
-) {
+): Error | undefined {
 	if (logger === undefined) {
 		return;
 	}

package/src/timeoutUtils.ts

@@ -19,31 +19,31 @@ class TestTimeout {
 	private deferred: Deferred<void> = new Deferred<void>();
 
 	private static instance: TestTimeout = new TestTimeout();
-	public static updateOnYield(runnable: Mocha.Runnable) {
+	public static updateOnYield(runnable: Mocha.Runnable): void {
 		TestTimeout.instance.clearTimer();
 		TestTimeout.instance.resetTimer(runnable);
 	}
 
-	public static reset() {
+	public static reset(): void {
 		TestTimeout.instance.clearTimer();
 		TestTimeout.instance = new TestTimeout();
 	}
 
-	public static getInstance() {
+	public static getInstance(): TestTimeout {
 		return TestTimeout.instance;
 	}
 
-	public async getPromise() {
+	public async getPromise(): Promise<void> {
 		return this.deferred.promise;
 	}
 
-	public getTimeout() {
+	public getTimeout(): number | undefined {
 		return this.timeout;
 	}
 
 	private constructor() {}
 
-	private resetTimer(runnable: Mocha.Runnable) {
+	private resetTimer(runnable: Mocha.Runnable): void {
 		assert(!this.timer, "clearTimer should have been called before reset");
 		assert(!this.deferred.isCompleted, "can't reset a completed TestTimeout");
 
@@ -61,7 +61,7 @@ class TestTimeout {
 			this.deferred.reject(this);
 		}, this.timeout);
 	}
-	private clearTimer() {
+	private clearTimer(): void {
 		if (this.timer) {
 			this.deferred = new Deferred();
 			clearTimeout(this.timer);
@@ -220,14 +220,14 @@ async function getTimeoutPromise<T = void>(
 	) => void,
 	timeoutOptions: TimeoutWithError | TimeoutWithValue<T>,
 	err: Error | undefined,
-) {
+): Promise<T> {
 	const timeout = timeoutOptions.durationMs ?? 0;
 	if (timeout <= 0 || !Number.isFinite(timeout)) {
 		return new Promise(executor);
 	}
 
 	return new Promise<T>((resolve, reject) => {
-		const timeoutRejections = () => {
+		const timeoutRejections = (): void => {
 			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
 			const errorObject = err!;
 			errorObject.message = `${errorObject.message} (${timeout}ms)`;