chore: tests for isSearchIndexSupported

Author: himanshusinghs

src/resources/common/debug.ts

@@ -62,7 +62,7 @@ export class DebugResource extends ReactiveResource<
         switch (this.current.tag) {
             case "connected": {
                 const searchIndexesSupported = await this.session.isSearchIndexSupported();
-                result += `The user is connected to the MongoDB cluster${searchIndexesSupported ? " with support for search indexes." : ""}.`;
+                result += `The user is connected to the MongoDB cluster${searchIndexesSupported ? " with support for search indexes" : ""}.`;
                 break;
             }
             case "errored":

tests/unit/resources/common/debug.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, it } from "vitest";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { DebugResource } from "../../../../src/resources/common/debug.js";
 import { Session } from "../../../../src/common/session.js";
 import { Telemetry } from "../../../../src/telemetry/telemetry.js";
@@ -13,13 +13,15 @@ import { Keychain } from "../../../../src/common/keychain.js";
 describe("debug resource", () => {
     const logger = new CompositeLogger();
     const deviceId = DeviceId.create(logger);
-    const session = new Session({
-        apiBaseUrl: "",
-        logger,
-        exportsManager: ExportsManager.init(config, logger),
-        connectionManager: new MCPConnectionManager(config, driverOptions, logger, deviceId),
-        keychain: new Keychain(),
-    });
+    const session = vi.mocked(
+        new Session({
+            apiBaseUrl: "",
+            logger,
+            exportsManager: ExportsManager.init(config, logger),
+            connectionManager: new MCPConnectionManager(config, driverOptions, logger, deviceId),
+            keychain: new Keychain(),
+        })
+    );
     const telemetry = Telemetry.create(session, { ...config, telemetry: "disabled" }, deviceId);
 
     let debugResource: DebugResource = new DebugResource(session, config, telemetry);
@@ -28,54 +30,54 @@ describe("debug resource", () => {
         debugResource = new DebugResource(session, config, telemetry);
     });
 
-    it("should be connected when a connected event happens", () => {
+    it("should be connected when a connected event happens", async () => {
         debugResource.reduceApply("connect", undefined);
-        const output = debugResource.toOutput();
+        const output = await debugResource.toOutput();
 
         expect(output).toContain(`The user is connected to the MongoDB cluster.`);
     });
 
-    it("should be disconnected when a disconnect event happens", () => {
+    it("should be disconnected when a disconnect event happens", async () => {
         debugResource.reduceApply("disconnect", undefined);
-        const output = debugResource.toOutput();
+        const output = await debugResource.toOutput();
 
         expect(output).toContain(`The user is not connected to a MongoDB cluster.`);
     });
 
-    it("should be disconnected when a close event happens", () => {
+    it("should be disconnected when a close event happens", async () => {
         debugResource.reduceApply("close", undefined);
-        const output = debugResource.toOutput();
+        const output = await debugResource.toOutput();
 
         expect(output).toContain(`The user is not connected to a MongoDB cluster.`);
     });
 
-    it("should be disconnected and contain an error when an error event occurred", () => {
+    it("should be disconnected and contain an error when an error event occurred", async () => {
         debugResource.reduceApply("connection-error", {
             tag: "errored",
             errorReason: "Error message from the server",
         });
 
-        const output = debugResource.toOutput();
+        const output = await debugResource.toOutput();
 
         expect(output).toContain(`The user is not connected to a MongoDB cluster because of an error.`);
         expect(output).toContain(`<error>Error message from the server</error>`);
     });
 
-    it("should show the inferred authentication type", () => {
+    it("should show the inferred authentication type", async () => {
         debugResource.reduceApply("connection-error", {
             tag: "errored",
             connectionStringAuthType: "scram",
             errorReason: "Error message from the server",
         });
 
-        const output = debugResource.toOutput();
+        const output = await debugResource.toOutput();
 
         expect(output).toContain(`The user is not connected to a MongoDB cluster because of an error.`);
         expect(output).toContain(`The inferred authentication mechanism is "scram".`);
         expect(output).toContain(`<error>Error message from the server</error>`);
     });
 
-    it("should show the atlas cluster information when provided", () => {
+    it("should show the atlas cluster information when provided", async () => {
         debugResource.reduceApply("connection-error", {
             tag: "errored",
             connectionStringAuthType: "scram",
@@ -88,7 +90,7 @@ describe("debug resource", () => {
             },
         });
 
-        const output = debugResource.toOutput();
+        const output = await debugResource.toOutput();
 
         expect(output).toContain(`The user is not connected to a MongoDB cluster because of an error.`);
         expect(output).toContain(
@@ -97,4 +99,12 @@ describe("debug resource", () => {
         expect(output).toContain(`The inferred authentication mechanism is "scram".`);
         expect(output).toContain(`<error>Error message from the server</error>`);
     });
+
+    it("should notify if a cluster supports search indexes", async () => {
+        session.isSearchIndexSupported = vi.fn().mockResolvedValue(true);
+        debugResource.reduceApply("connect", undefined);
+        const output = await debugResource.toOutput();
+
+        expect(output).toContain(`The user is connected to the MongoDB cluster with support for search indexes.`);
+    });
 });