fix: add more test coverage

Author: shanejonas

src/coverage.test.ts

@@ -27,6 +27,7 @@ const mockSchema = {
     },
     {
       name: "bar",
+      paramStructure: "by-name",
       params: [
         {
           name: "barParam",
@@ -49,6 +50,54 @@ const mockSchema = {
           type: "boolean",
         },
       },
+      examples: [
+        {
+          name: "fooExample",
+          summary: "foo example",
+          description: "this is an example of foo",
+          params: [
+            {
+              name: "barParam",
+              value: "bar",
+            },
+            {
+              name: "barParam2",
+              value: "bar",
+            }
+          ],
+          result: {
+            name: "fooResult",
+            schema: {
+              type: "boolean",
+            },
+          },
+        },
+      ],
+    },
+    {
+      name: "baz",
+      params: [
+        {
+          name: "bazParam",
+          required: true,
+          schema: {
+            type: "string",
+            enum: ["baz"],
+          },
+        },
+        {
+          name: "bazParam2",
+          schema: {
+            type: "string",
+          },
+        },
+      ],
+      result: {
+        name: "bazResult",
+        schema: {
+          type: "boolean",
+        },
+      },
       examples: [
         {
           name: "fooExample",
@@ -109,6 +158,29 @@ describe("coverage", () => {
       });
     });
   });
+  describe("coverage tests", () => {
+    it("throws an error when there are no methods", async () => {
+      const reporter = new class CustomReporter {
+        onBegin() {}
+        onTestBegin() {}
+        onTestEnd() {}
+        onEnd() {}
+      };
+      const spy = jest.spyOn(reporter, "onTestBegin");
+      const transport = () => Promise.resolve({});
+      const openrpcDocument = mockSchema;
+      const options = {
+        reporter,
+        transport,
+        openrpcDocument,
+        skip: ['foo', 'bar', 'baz'],
+        only: [],
+      };
+
+      await coverage(options);
+      expect(spy).toHaveBeenCalledTimes(0);
+    });
+  });
   describe("transport", () => {
     it("can call the transport", (done) => {
       const transport = () => {
@@ -124,7 +196,7 @@ describe("coverage", () => {
       });
     });
   });
-  describe("reporter", () => {
+  describe("reporter more tests", () => {
     // reporter integration tests
     it("onBegin is called", async () => {
       // this is a test that the reporter is called
@@ -162,7 +234,7 @@ describe("coverage", () => {
       };
 
       await coverage(options);
-      expect(spy).toHaveBeenCalledTimes(11);
+      expect(spy).toHaveBeenCalledTimes(12);
     });
   });
 });

src/coverage.ts

@@ -68,6 +68,7 @@ export default async (options: IOptions) => {
   servers.forEach(({ url }) => {
     filteredMethods.forEach((method) => {
       if (method.examples === undefined || method.examples.length === 0) {
+        console.log("method : ", method);
         for (let i = 0; i < 10; i++) {
           const p = getFakeParams(method.params);
           // handle object or array case
@@ -92,6 +93,7 @@ export default async (options: IOptions) => {
           method.paramStructure === "by-name"
             ? paramsToObj(p, method.params as ContentDescriptorObject[])
             : p;
+        console.log("params : ", params);
         exampleCalls.push({
           title: method.name + " > example params and expect result to match: " + ex.name,
           methodName: method.name,