@fluidframework/test-utils 2.0.0-internal.6.4.0 → 2.0.0-internal.7.0.0

package/src/testContainerRuntimeFactory.ts

@@ -3,6 +3,7 @@
  * Licensed under the MIT License.
  */
 
+// eslint-disable-next-line import/no-deprecated
 import { defaultRouteRequestHandler } from "@fluidframework/aqueduct";
 import { IContainerContext, IRuntime } from "@fluidframework/container-definitions";
 import {
@@ -11,6 +12,7 @@ import {
 	DefaultSummaryConfiguration,
 } from "@fluidframework/container-runtime";
 import { IContainerRuntime } from "@fluidframework/container-runtime-definitions";
+// eslint-disable-next-line import/no-deprecated
 import { buildRuntimeRequestHandler, RuntimeRequestHandler } from "@fluidframework/request-handler";
 import { IFluidDataStoreFactory } from "@fluidframework/runtime-definitions";
 import { RuntimeFactoryHelper } from "@fluidframework/runtime-utils";
@@ -52,8 +54,7 @@ export const createTestContainerRuntimeFactory = (
 		public async instantiateFromExisting(runtime: ContainerRuntime): Promise<void> {
 			// Validate we can load root data stores.
 			// We should be able to load any data store that was created in initializeFirstTime!
-			await (runtime.getAliasedDataStoreEntryPoint?.("default") ??
-				runtime.getRootDataStore("default"));
+			await runtime.getAliasedDataStoreEntryPoint("default");
 		}
 
 		async preInitialize(
@@ -66,7 +67,9 @@ export const createTestContainerRuntimeFactory = (
 					["default", Promise.resolve(this.dataStoreFactory)],
 					[this.type, Promise.resolve(this.dataStoreFactory)],
 				],
+				// eslint-disable-next-line import/no-deprecated
 				buildRuntimeRequestHandler(
+					// eslint-disable-next-line import/no-deprecated
 					defaultRouteRequestHandler("default"),
 					...this.requestHandlers,
 				),

package/src/testContainerRuntimeFactoryWithDefaultDataStore.ts

@@ -0,0 +1,50 @@
+/*!
+ * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
+ * Licensed under the MIT License.
+ */
+
+import { ContainerRuntimeFactoryWithDefaultDataStore } from "@fluidframework/aqueduct";
+import { IContainerRuntimeOptions } from "@fluidframework/container-runtime";
+import { IContainerRuntime } from "@fluidframework/container-runtime-definitions";
+import { FluidObject } from "@fluidframework/core-interfaces";
+import { RuntimeRequestHandler } from "@fluidframework/request-handler";
+import {
+	IFluidDataStoreFactory,
+	NamedFluidDataStoreRegistryEntries,
+} from "@fluidframework/runtime-definitions";
+
+/**
+ * ! Note: This function is purely needed for back-compat as the constructor argument structure was changed
+ */
+export const createContainerRuntimeFactoryWithDefaultDataStore = (
+	Base: typeof ContainerRuntimeFactoryWithDefaultDataStore = ContainerRuntimeFactoryWithDefaultDataStore,
+	ctorArgs: {
+		defaultFactory: IFluidDataStoreFactory;
+		registryEntries: NamedFluidDataStoreRegistryEntries;
+		dependencyContainer?: any;
+		requestHandlers?: RuntimeRequestHandler[];
+		runtimeOptions?: IContainerRuntimeOptions;
+		provideEntryPoint?: (runtime: IContainerRuntime) => Promise<FluidObject>;
+	},
+): ContainerRuntimeFactoryWithDefaultDataStore => {
+	try {
+		return new Base(ctorArgs);
+	} catch (err) {
+		// IMPORTANT: The constructor argument structure changed, so this is needed for dynamically using older ContainerRuntimeFactoryWithDefaultDataStore's
+		const {
+			defaultFactory,
+			registryEntries,
+			dependencyContainer,
+			requestHandlers,
+			runtimeOptions,
+		} = ctorArgs;
+		// eslint-disable-next-line @typescript-eslint/no-unsafe-return
+		return new (Base as any)(
+			defaultFactory,
+			registryEntries,
+			dependencyContainer,
+			requestHandlers,
+			runtimeOptions,
+		);
+	}
+};

package/src/testFluidObject.ts

@@ -3,6 +3,7 @@
  * Licensed under the MIT License.
  */
 
+// eslint-disable-next-line import/no-deprecated
 import { defaultFluidObjectRequestHandler } from "@fluidframework/aqueduct";
 import {
 	IRequest,
@@ -94,6 +95,7 @@ export class TestFluidObject implements ITestFluidObject {
 	 * @deprecated - Will be removed in future major release. Migrate all usage of IFluidRouter to the "entryPoint" pattern. Refer to Removing-IFluidRouter.md
 	 */
 	public async request(request: IRequest): Promise<IResponse> {
+		// eslint-disable-next-line import/no-deprecated
 		return defaultFluidObjectRequestHandler(this, request);
 	}
 
@@ -200,10 +202,9 @@ export class TestFluidObjectFactory implements IFluidDataStoreFactory {
 
 		const runtimeClass = mixinRequestHandler(
 			async (request: IRequest, rt: FluidDataStoreRuntime) => {
-				const maybeRouter: FluidObject<IProvideFluidRouter> | undefined =
-					await rt.entryPoint?.get();
+				const maybeRouter: FluidObject<IProvideFluidRouter> = await rt.entryPoint.get();
 				assert(
-					maybeRouter?.IFluidRouter !== undefined,
+					maybeRouter.IFluidRouter !== undefined,
 					"entryPoint should have been initialized by now",
 				);
 				return maybeRouter.IFluidRouter.request(request);

package/src/testObjectProvider.ts

@@ -7,7 +7,6 @@ import {
 	IContainer,
 	IHostLoader,
 	IFluidCodeDetails,
-	LoaderHeader,
 	ILoader,
 } from "@fluidframework/container-definitions";
 import {
@@ -112,9 +111,6 @@ export interface ITestContainerConfig {
 
 	/** Loader options for the loader used to create containers */
 	loaderProps?: Partial<ILoaderProps>;
-
-	/** Temporary flag: simulate read connection using delay connection, default is true */
-	simulateReadConnectionUsingDelay?: boolean;
 }
 
 export const createDocumentId = (): string => uuid();
@@ -386,41 +382,11 @@ export class TestObjectProvider implements ITestObjectProvider {
 		return this.resolveContainer(loader, requestHeader);
 	}
 
-	private async resolveContainer(
-		loader: ILoader,
-		requestHeader?: IRequestHeader,
-		delay: boolean = true,
-	) {
-		// Once AB#3889 is done to switch default connection mode to "read" on load, we don't need
-		// to load "delayed" across the board. Remove the following code.
-		const delayConnection =
-			delay &&
-			(requestHeader === undefined || requestHeader[LoaderHeader.reconnect] !== false);
-		const headers: IRequestHeader | undefined = delayConnection
-			? {
-					[LoaderHeader.loadMode]: { deltaConnection: "delayed" },
-					...requestHeader,
-			  }
-			: requestHeader;
-
-		const container = await loader.resolve({
+	private async resolveContainer(loader: ILoader, headers?: IRequestHeader) {
+		return loader.resolve({
 			url: await this.driver.createContainerUrl(this.documentId),
 			headers,
 		});
-
-		// Once AB#3889 is done to switch default connection mode to "read" on load, we don't need
-		// to load "delayed" across the board. Remove the following code.
-		if (delayConnection) {
-			// Older version may not have connect/disconnect. It was add in PR#9439, and available >= 0.59.1000
-			const maybeContainer = container as Partial<IContainer>;
-			if (maybeContainer.connect !== undefined) {
-				container.connect();
-			} else {
-				// back compat. Remove when we don't support < 0.59.1000
-				(container as any).resume();
-			}
-		}
-		return container;
 	}
 
 	/**
@@ -473,11 +439,7 @@ export class TestObjectProvider implements ITestObjectProvider {
 	): Promise<IContainer> {
 		const loader = this.makeTestLoader(testContainerConfig);
 
-		const container = await this.resolveContainer(
-			loader,
-			requestHeader,
-			testContainerConfig?.simulateReadConnectionUsingDelay,
-		);
+		const container = await this.resolveContainer(loader, requestHeader);
 		await this.waitContainerToCatchUp(container);
 
 		return container;