Assert all required params in the contract test

Author: sergey-ermakovich

test/tools.test.mjs

@@ -18,7 +18,7 @@ const KEY = process.env.HASDATA_API_KEY;
 const TIMEOUT_MS = 30_000;
 
 const EXPECTED = {
-    hasdata_google_travel_flights_getGoogleFlights: 'departureId',
+    hasdata_google_travel_flights_getGoogleFlights: ['departureId', 'arrivalId', 'outboundDate'],
 };
 
 // A streamable HTTP body arrives either as plain JSON or as server-sent events. One SSE event
@@ -105,10 +105,12 @@ test('the tool still declares its required parameter', live, async () => {
         const required = tool.inputSchema?.required ?? [];
         const want = EXPECTED[tool.name];
         assert.ok(want, `tool ${tool.name} is not covered by this test`);
-        assert.ok(
-            required.includes(want),
-            `${tool.name} should require ${want}, declares: ${required.join(', ') || 'nothing'}`
-        );
+        for (const param of want) {
+            assert.ok(
+                required.includes(param),
+                `${tool.name} should require ${param}, declares: ${required.join(', ') || 'nothing'}`
+            );
+        }
     }
 });