test: review updates

Author: cdcabrera

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

@@ -6,6 +6,9 @@ exports[`runServer should allow server to be stopped, http stop server: diagnost
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "Registered tool: usePatternFlyDocs",
     ],
@@ -35,6 +38,9 @@ exports[`runServer should allow server to be stopped, stdio stop server: diagnos
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "Registered tool: usePatternFlyDocs",
     ],
@@ -64,6 +70,9 @@ exports[`runServer should attempt to run server, create transport, connect, and
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "test-server-4 server running on stdio transport",
     ],
@@ -98,6 +107,9 @@ exports[`runServer should attempt to run server, disable SIGINT handler: diagnos
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "test-server-7 server running on stdio transport",
     ],
@@ -127,6 +139,9 @@ exports[`runServer should attempt to run server, enable SIGINT handler explicitl
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "test-server-8 server running on stdio transport",
     ],
@@ -161,6 +176,9 @@ exports[`runServer should attempt to run server, register a tool: diagnostics 1`
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "Registered tool: loremIpsum",
     ],
@@ -203,6 +221,9 @@ exports[`runServer should attempt to run server, register multiple tools: diagno
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "Registered tool: loremIpsum",
     ],
@@ -252,6 +273,9 @@ exports[`runServer should attempt to run server, use custom options: diagnostics
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "test-server-3 server running on stdio transport",
     ],
@@ -286,6 +310,9 @@ exports[`runServer should attempt to run server, use default tools, http: diagno
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "Registered tool: usePatternFlyDocs",
     ],
@@ -333,6 +360,9 @@ exports[`runServer should attempt to run server, use default tools, stdio: diagn
     [
       "Server logging enabled.",
     ],
+    [
+      "No external tools loaded.",
+    ],
     [
       "Registered tool: usePatternFlyDocs",
     ],

tests/__fixtures__/tool.echo.js

@@ -18,4 +18,4 @@ const echo_plugin_tool = options => [
   })
 ];
 
-export { echo_plugin_tool as default, echo_plugin_tool };
+export default echo_plugin_tool;

tests/__snapshots__/stdioTransport.test.ts.snap

@@ -271,6 +271,8 @@ exports[`Logging should allow setting logging options, default 1`] = `[]`;
 exports[`Logging should allow setting logging options, stderr 1`] = `
 [
   "[INFO]: Server logging enabled.
+",
+  "[INFO]: No external tools loaded.
 ",
   "[INFO]: Registered tool: usePatternFlyDocs
 ",

tests/stdioTransport.test.ts

@@ -196,8 +196,8 @@ describe('Tools', () => {
     const resp = await CLIENT.send(req);
     const names = (resp?.result?.tools ?? []).map((tool: any) => tool.name);
 
-    expect(names).toContain('echo_plugin_tool');
     expect(CLIENT.logs()).toMatchSnapshot();
+    expect(names).toContain('echo_plugin_tool');
   });
 
   it('should interact with the new tool', async () => {