refactored the httpclients to remove the duplication after introducin…

…g the basic http-client. Adding timeout error

axios/http-json-client.js

@@ -1,33 +1,12 @@
-const http = require('http')
-const https = require('https')
-const axios = require('axios')
+const httpClient = require('./http-client')
 
-module.exports = ({
-  baseURL,
-  timeout = 2500,
-  secure = true,
-  address = '0.0.0.0',
-  hostname = address || '0.0.0.0',
-  port = secure === true ? 443 : 80,
-  rejectUnauthorized = secure !== false,
-  protocol = secure === true ? 'https' : 'http',
-  httpsAgent = new https.Agent({keepAlive: true, rejectUnauthorized}),
-  httpAgent = new http.Agent({keepAlive: true}),
-  headers = {},
-} = {}) => axios.create({
-  baseURL: baseURL || `${protocol}://${hostname}:${port}`,
-  timeout,
-  httpAgent,
-  httpsAgent,
-  transitional: {
-    clarifyTimeoutError: true,
-    // forcedJSONParsing: false,
-    silentJSONParsing: false,
-  },
+module.exports = ({headers = {}, ...options} = {}) => httpClient({
+  timeout: 2500,
   headers: {
     'Accept': 'application/json',
     'Accept-Encoding': 'gzip, deflate',
     'Content-Type': 'application/json; charset=utf-8',
     ...headers,
   },
+  ...options,
 })

tests/http/http-timeout.test.js

@@ -0,0 +1,6 @@
+const {client} = global.t
+
+test('basic http test', async () => {
+  const {status} = await client({timeout: 100}).post('/long', {})
+  expect(status).toEqual(200)
+})

tests/http/routes.js

@@ -1,3 +1,4 @@
+const {setTimeout: sleep} = require('timers/promises')
 const {defaultHttpJsonClient} = require('../../axios')
 const {__} = require('../../fusto/')
 
@@ -35,6 +36,11 @@ module.exports = app => {
     await get('/')
   })
 
+  app.post('/long', async () => {
+    await sleep(400)
+    return {slow: true}
+  })
+
   app.get('/async-pipe-booom', async (request, reply) => {
     await __([1])
       // eslint-disable-next-line require-await