refactor(options): mode option for e2e testing

Author: cdcabrera

src/__tests__/__snapshots__/options.defaults.test.ts.snap

@@ -5,6 +5,7 @@ exports[`options defaults should return specific properties: defaults 1`] = `
   "contextPath": "/",
   "contextUrl": "file:///",
   "docsPath": "/documentation",
+  "docsPathSlug": "documentation:",
   "http": {
     "allowedHosts": [],
     "allowedOrigins": [],
@@ -21,8 +22,27 @@ exports[`options defaults should return specific properties: defaults 1`] = `
   },
   "maxDocsToLoad": 500,
   "maxSearchLength": 256,
+  "mode": "programmatic",
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "name": "@patternfly/patternfly-mcp",
   "nodeVersion": 22,
+  "patternflyOptions": {
+    "availableResourceVersions": [
+      "v6",
+    ],
+    "default": {
+      "defaultVersion": "v6",
+      "versionStrategy": "highest",
+      "versionWhitelist": [
+        "@patternfly/react-core",
+        "@patternfly/patternfly",
+      ],
+    },
+  },
   "pfExternal": "https://raw.githubusercontent.com/patternfly/patternfly-org/fb05713aba75998b5ecf5299ee3c1a259119bd74/packages/documentation-site/patternfly-docs/content",
   "pfExternalAccessibility": "https://raw.githubusercontent.com/patternfly/patternfly-org/fb05713aba75998b5ecf5299ee3c1a259119bd74/packages/documentation-site/patternfly-docs/content/accessibility",
   "pfExternalChartsDesign": "https://raw.githubusercontent.com/patternfly/patternfly-org/fb05713aba75998b5ecf5299ee3c1a259119bd74/packages/documentation-site/patternfly-docs/content/design-guidelines/charts",

src/__tests__/__snapshots__/options.test.ts.snap

@@ -16,6 +16,11 @@ exports[`parseCliOptions should attempt to parse args with --allowed-hosts 1`] =
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -37,6 +42,11 @@ exports[`parseCliOptions should attempt to parse args with --allowed-origins 1`]
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -55,6 +65,11 @@ exports[`parseCliOptions should attempt to parse args with --http and --host 1`]
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -73,6 +88,11 @@ exports[`parseCliOptions should attempt to parse args with --http and --port 1`]
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -91,6 +111,11 @@ exports[`parseCliOptions should attempt to parse args with --http and invalid --
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -107,6 +132,11 @@ exports[`parseCliOptions should attempt to parse args with --http flag 1`] = `
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -123,6 +153,11 @@ exports[`parseCliOptions should attempt to parse args with --log-level flag 1`]
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -139,6 +174,11 @@ exports[`parseCliOptions should attempt to parse args with --log-stderr flag and
     "stderr": true,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -155,6 +195,11 @@ exports[`parseCliOptions should attempt to parse args with --tool 1`] = `
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [
     "my-tool",
@@ -174,6 +219,11 @@ exports[`parseCliOptions should attempt to parse args with --verbose flag 1`] =
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -190,6 +240,11 @@ exports[`parseCliOptions should attempt to parse args with --verbose flag and --
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }
@@ -206,6 +261,11 @@ exports[`parseCliOptions should attempt to parse args with other arguments 1`] =
     "stderr": false,
     "transport": "stdio",
   },
+  "modeOptions": {
+    "cli": {},
+    "programmatic": {},
+    "test": {},
+  },
   "pluginIsolation": undefined,
   "toolModules": [],
 }

src/__tests__/__snapshots__/server.getResources.test.ts.snap

@@ -93,6 +93,8 @@ exports[`promiseQueue should execute promises in order: allSettled 1`] = `
 
 exports[`resolveLocalPathFunction should return a consistent path, basic 1`] = `"/lorem-ipsum.md"`;
 
+exports[`resolveLocalPathFunction should return a consistent path, documentation slug 1`] = `"/documentation/guidelines/README.md"`;
+
 exports[`resolveLocalPathFunction should return a consistent path, url, file 1`] = `"file://someDirectory/dolor-sit.md"`;
 
 exports[`resolveLocalPathFunction should return a consistent path, url, http 1`] = `"http://example.com/dolor-sit.md"`;

src/__tests__/index.test.ts

@@ -79,7 +79,7 @@ describe('main', () => {
 
     mockRunServer.mockRejectedValue(error);
 
-    await main();
+    await expect(async () => main()).rejects.toThrow(error.message);
 
     expect(consoleErrorSpy).toHaveBeenCalledWith('Failed to start server:', error);
     expect(processExitSpy).toHaveBeenCalledWith(1);
@@ -89,33 +89,33 @@ describe('main', () => {
     {
       description: 'parseCliOptions',
       error: new Error('Failed to parse CLI options'),
-      message: 'Failed to start server:',
+      message: 'Set options error, failed to start server:',
       method: main
     },
     {
       description: 'setOptions',
       error: new Error('Failed to set options'),
-      message: 'Failed to start server:',
+      message: 'Set options error, failed to start server:',
       method: main
     },
     {
       description: 'parseCliOptions, with start alias',
       error: new Error('Failed to parse CLI options'),
-      message: 'Failed to start server:',
+      message: 'Set options error, failed to start server:',
       method: start
     },
     {
       description: 'setOptions, with start alias',
       error: new Error('Failed to set options'),
-      message: 'Failed to start server:',
+      message: 'Set options error, failed to start server:',
       method: start
     }
   ])('should handle errors, $description', async ({ error, message, method }) => {
     mockSetOptions.mockImplementation(() => {
       throw error;
     });
 
-    await method();
+    await expect(async () => method()).rejects.toThrow(error.message);
 
     expect(consoleErrorSpy).toHaveBeenCalledWith(message, error);
     expect(processExitSpy).toHaveBeenCalledWith(1);