Merge pull request #3 from open-rpc/fix/transports-and-coverage

fix: add server support and update readme with install and usage

Author: shanejonas

.gitignore

@@ -1,2 +1,3 @@
 node_modules
 build
+*openrpc.json

README.md

@@ -15,9 +15,26 @@ Even after writing your OpenRPC Document, you want to test that the OpenRPC Docu
 - report back the coverage results
 - show errors and the result to help fix inconsistencies
 
+### Installation:
+
+```
+
+npm install -g @open-rpc/test-coverage
+
+```
+
+
+### Usage:
+
+
+```
+open-rpc-test-coverage -s https://raw.githubusercontent.com/open-rpc/examples/master/service-descriptions/simple-math-openrpc.json --transport=http --reporter=console --skipMethods=addition
+```
+
+
 
 #### Screenshot
 
 ##### console reporter
 
-![image](https://user-images.githubusercontent.com/364566/56315309-377dbd80-610c-11e9-8be3-dd99f6325461.png)
+![image](https://user-images.githubusercontent.com/364566/56318521-3e103300-6114-11e9-85cd-f35eb7b42a0e.png)

src/coverage.test.ts

@@ -1,14 +1,18 @@
 import coverage from "./coverage";
 import { OpenRPC } from "@open-rpc/meta-schema";
 
-
-
 const mockSchema = {
   openrpc: "1.0.0",
   info: {
     title: "my api",
     version: "0.0.0-development"
   },
+  servers: [
+    {
+      name: 'my api',
+      url: 'http://localhost:3333'
+    }
+  ],
   methods: [
     {
       name: 'foo',
@@ -23,7 +27,6 @@ const mockSchema = {
   ]
 } as OpenRPC
 
-
 describe('coverage', () => {
   it('can call the reporter', (done) => {
     const reporter = () => done()

src/coverage.ts

@@ -14,7 +14,7 @@ const results: any[] = [];
 interface IOptions {
   schema: OpenRPC;
   skipMethods: string[];
-  transport(method: string, params: any[]): PromiseLike<any>;
+  transport(url: string, method: string, params: any[]): PromiseLike<any>;
   reporter(value: any[]): any;
 }
 
@@ -24,25 +24,31 @@ export default async (options: IOptions) => {
       return Promise.resolve();
     }
     const params = await getParams(method.params);
-    return options.transport(method.name, params)
-      .then((r: any) => {
-        if (r.error && r.error.message.includes('does not exist')) {
-          results.push({
-            method: method.name,
-            error: 'Method Does Not Exist'
-          });
-        } else if (r.error) {
-          results.push({
-            method: method.name,
-            error: r.error.message
-          });
-        } else {
-          results.push({
-            method: method.name,
-            result: r.result
-          });
-        }
-      });
+    const urls = (options.schema.servers || []).map((u) => {
+      // TODO: support server variables
+      return u.url;
+    })
+    return Promise.all(urls.map((url) => {
+      return options.transport(url, method.name, params)
+        .then((r: any) => {
+          if (r.error && r.error.message.includes('does not exist')) {
+            results.push({
+              method: method.name,
+              error: 'Method Does Not Exist'
+            });
+          } else if (r.error) {
+            results.push({
+              method: method.name,
+              error: r.error.message
+            });
+          } else {
+            results.push({
+              method: method.name,
+              result: r.result
+            });
+          }
+        });
+    }))
   });
 
   return Promise.all(promises).then(() => {

src/transports/HTTPTransport.ts

@@ -1,11 +1,12 @@
 import fetch from 'isomorphic-fetch';
 let id = 1;
 
-export default (method: string, params: any[]) => {
-  return fetch('http://localhost:8545', {
+export default (url: string, method: string, params: any[]) => {
+  return fetch(url, {
     method: "POST",
     headers: { "Content-Type": "application/json" },
     body: JSON.stringify({
+      jsonrpc: "2.0",
       id: id++,
       method,
       params