removing request and request-promise dependencies (#9)

Co-authored-by: Ivan Martinez Pupo <[email protected]>

helpers/schema-loader.js

@@ -1,6 +1,6 @@
 const { createSchema } = require('swagger-to-graphql')
 const callBackend = require('./callbackend')
-const request = require('request-promise')
+const fetch = require('node-fetch')
 
 /**
  * @param {GraqhQLSchema} localSchema
@@ -9,11 +9,14 @@ const request = require('request-promise')
  */
 module.exports = async ({ localSchema, remoteRestServices }) => {
   const schemasAvailable = remoteRestServices.map(async service => {
-    const swaggerResponse = await request({
-      uri: service.url,
-      headers: service.headers
+    const swaggerResponse = await fetch(service.url, {
+      headers: {
+        'Content-Type': 'application/json',
+        ...service.headers
+      }
     })
-    let swaggerSchema = JSON.parse(swaggerResponse)
+
+    let swaggerSchema = await swaggerResponse.json()
 
     if (service.onLoaded) {
       swaggerSchema = service.onLoaded(swaggerSchema, service)

package.json

@@ -11,8 +11,6 @@
     "graphql": "^14.5.8",
     "graphql-tag": "^2.10.1",
     "graphql-tools": "^4.0.6",
-    "request": "^2.88.0",
-    "request-promise": "^4.2.5",
     "swagger-to-graphql": "^4.0.2",
     "node-fetch": "^2.6.0"
   },
@@ -50,5 +48,5 @@
   "standard": {
     "env": [ "jest" ]
   },
-  "version": "1.0.6"
+  "version": "1.0.7"
 }

test/construct.spec.js

@@ -15,7 +15,8 @@ describe('Server construct', () => {
       const endpointsList = [{ name: 'petstore_service', url: 'https://no-existent-endpoint/swagger.json' }]
       await gateway({ endpointsList })
     } catch (err) {
-      expect(err.message).toBe('Error: getaddrinfo ENOTFOUND no-existent-endpoint no-existent-endpoint:443')
+      expect(err.code).toBe('ENOTFOUND')
+      expect(err.message).toBe('request to https://no-existent-endpoint/swagger.json failed, reason: getaddrinfo ENOTFOUND no-existent-endpoint no-existent-endpoint:443')
     }
   })