@bryan-thompson/inspector-assessment-cli 1.27.0 → 1.29.0

package/build/__tests__/flag-parsing.test.js

@@ -11,7 +11,9 @@
  * - Profile/format validation - CLI option validation
  * - Mutual exclusivity - Flag conflict detection
  */
-import { describe, it, expect } from "@jest/globals";
+import { describe, it, expect, jest } from "@jest/globals";
+import { parseArgs, printVersion } from "../lib/cli-parser.js";
+import packageJson from "../../package.json" with { type: "json" };
 describe("Key-Value Pair Parsing", () => {
     /**
      * Recreates parseKeyValuePair logic from cli.ts
@@ -603,3 +605,42 @@ describe("Log Level Validation", () => {
         });
     });
 });
+describe("Version Flag Parsing", () => {
+    describe("parseArgs with version flags", () => {
+        it("should set versionRequested flag for --version", () => {
+            const options = parseArgs(["test-server", "--version"]);
+            expect(options.versionRequested).toBe(true);
+        });
+        it("should set versionRequested flag for -V", () => {
+            const options = parseArgs(["test-server", "-V"]);
+            expect(options.versionRequested).toBe(true);
+        });
+        it("should return early when version flag is present", () => {
+            const options = parseArgs(["--version"]);
+            expect(options.versionRequested).toBe(true);
+            expect(options.serverName).toBeUndefined();
+        });
+        it("should handle version flag at any position", () => {
+            const options = parseArgs(["--version", "--verbose"]);
+            expect(options.versionRequested).toBe(true);
+        });
+    });
+    describe("printVersion function", () => {
+        it("should output version in correct format", () => {
+            const consoleSpy = jest
+                .spyOn(console, "log")
+                .mockImplementation(() => { });
+            printVersion();
+            expect(consoleSpy).toHaveBeenCalledWith(expect.stringMatching(/^mcp-assess-full \d+\.\d+\.\d+$/));
+            consoleSpy.mockRestore();
+        });
+        it("should match package.json version", () => {
+            const consoleSpy = jest
+                .spyOn(console, "log")
+                .mockImplementation(() => { });
+            printVersion();
+            expect(consoleSpy).toHaveBeenCalledWith(`mcp-assess-full ${packageJson.version}`);
+            consoleSpy.mockRestore();
+        });
+    });
+});

package/build/__tests__/http-transport-integration.test.js

@@ -25,6 +25,8 @@ const DEFAULT_HEADERS = {
  */
 async function checkServerAvailable(url) {
     try {
+        const controller = new AbortController();
+        const timeoutId = setTimeout(() => controller.abort(), 5000);
         const response = await fetch(url, {
             method: "POST",
             headers: DEFAULT_HEADERS,
@@ -38,7 +40,9 @@ async function checkServerAvailable(url) {
                 },
                 id: 1,
             }),
+            signal: controller.signal,
         });
+        clearTimeout(timeoutId);
         // Accept any response (200 or error) as indication server is up
         return response.status < 500;
     }
@@ -108,7 +112,7 @@ describe("HTTP Transport Integration", () => {
             console.log("   - vulnerable-mcp: http://localhost:10900/mcp");
             console.log("   - hardened-mcp: http://localhost:10901/mcp\n");
         }
-    });
+    }, 30000);
     describe("HTTP Transport Creation (Unit-level)", () => {
         it("should create transport with valid HTTP URL", () => {
             const options = {

package/build/__tests__/testbed-integration.test.js

@@ -27,6 +27,8 @@ const DEFAULT_HEADERS = {
  */
 async function checkServerAvailable(url) {
     try {
+        const controller = new AbortController();
+        const timeoutId = setTimeout(() => controller.abort(), 5000);
         const response = await fetch(url, {
             method: "POST",
             headers: DEFAULT_HEADERS,
@@ -40,7 +42,9 @@ async function checkServerAvailable(url) {
                 },
                 id: 1,
             }),
+            signal: controller.signal,
         });
+        clearTimeout(timeoutId);
         return response.status < 500;
     }
     catch {
@@ -133,7 +137,7 @@ describe("Testbed A/B Comparison", () => {
             console.log("   - vulnerable-mcp: http://localhost:10900/mcp");
             console.log("   - hardened-mcp: http://localhost:10901/mcp\n");
         }
-    });
+    }, 30000);
     describe("Health Check Tests", () => {
         it("should connect to vulnerable-mcp server", async () => {
             if (!vulnerableAvailable) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bryan-thompson/inspector-assessment-cli",
-  "version": "1.27.0",
+  "version": "1.29.0",
   "description": "CLI for the Enhanced MCP Inspector with assessment capabilities",
   "license": "MIT",
   "author": "Bryan Thompson <bryan@triepod.ai>",