test: replace t.type with t.ok and instanceof

test/client-connect.js

@@ -31,7 +31,7 @@ test('connect aborted after connect', (t) => {
       opaque: 'asd'
     }, (err, { opaque }) => {
       t.equal(opaque, 'asd')
-      t.type(err, errors.RequestAbortedError)
+      t.ok(err instanceof errors.RequestAbortedError)
     })
     t.equal(client[kBusy], true)
   })

test/client-pipeline.js

@@ -340,7 +340,7 @@ test('pipeline invalid handler return', (t) => {
       body.on('error', () => {})
     })
       .on('error', (err) => {
-        t.type(err, errors.InvalidReturnValueError)
+        t.ok(err instanceof errors.InvalidReturnValueError)
       })
       .end()
 
@@ -353,7 +353,7 @@ test('pipeline invalid handler return', (t) => {
       return {}
     })
       .on('error', (err) => {
-        t.type(err, errors.InvalidReturnValueError)
+        t.ok(err instanceof errors.InvalidReturnValueError)
       })
       .end()
   })
@@ -409,7 +409,7 @@ test('pipeline destroy and throw handler', (t) => {
     })
       .end()
       .on('error', (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
       })
       .on('close', () => {
         t.pass()
@@ -449,7 +449,7 @@ test('pipeline abort res', (t) => {
       return body
     })
       .on('error', (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
       })
       .end()
   })
@@ -474,7 +474,7 @@ test('pipeline abort server res', (t) => {
       t.fail()
     })
       .on('error', (err) => {
-        t.type(err, errors.SocketError)
+        t.ok(err instanceof errors.SocketError)
       })
       .end()
   })
@@ -505,7 +505,7 @@ test('pipeline abort duplex', (t) => {
       }, () => {
         t.fail()
       }).destroy().on('error', (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
       })
     })
   })
@@ -558,7 +558,7 @@ test('pipeline abort piped res 2', (t) => {
     }, ({ body }) => {
       const pt = new PassThrough()
       body.on('error', (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
       })
       setImmediate(() => {
         pt.destroy()
@@ -567,7 +567,7 @@ test('pipeline abort piped res 2', (t) => {
       return pt
     })
       .on('error', (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
       })
       .end()
   })
@@ -628,7 +628,7 @@ test('pipeline abort server res after headers', (t) => {
       return data.body
     })
       .on('error', (err) => {
-        t.type(err, errors.SocketError)
+        t.ok(err instanceof errors.SocketError)
       })
       .end()
   })
@@ -656,7 +656,7 @@ test('pipeline w/ write abort server res after headers', (t) => {
       return data.body
     })
       .on('error', (err) => {
-        t.type(err, errors.SocketError)
+        t.ok(err instanceof errors.SocketError)
       })
       .resume()
       .write('asd')
@@ -713,7 +713,7 @@ test('pipeline args validation', (t) => {
   const ret = client.pipeline(null, () => {})
   ret.on('error', (err) => {
     t.ok(/opts/.test(err.message))
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 })
 
@@ -866,7 +866,7 @@ test('pipeline CONNECT throw', (t) => {
     }, () => {
       t.fail()
     }).on('error', (err) => {
-      t.type(err, errors.InvalidArgumentError)
+      t.ok(err instanceof errors.InvalidArgumentError)
     })
     client.on('disconnect', () => {
       t.fail()
@@ -1030,7 +1030,7 @@ test('pipeline abort after headers', (t) => {
     })
       .end()
       .on('error', (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
       })
   })
 })

test/client-request.js

@@ -117,7 +117,7 @@ test('request abort before headers', (t) => {
         method: 'GET',
         signal
       }, (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
         t.equal(signal.listenerCount('abort'), 0)
       })
       t.equal(signal.listenerCount('abort'), 1)
@@ -127,7 +127,7 @@ test('request abort before headers', (t) => {
         method: 'GET',
         signal
       }, (err) => {
-        t.type(err, errors.RequestAbortedError)
+        t.ok(err instanceof errors.RequestAbortedError)
         t.equal(signal.listenerCount('abort'), 0)
       })
       t.equal(signal.listenerCount('abort'), 2)

test/client-stream.js

@@ -263,17 +263,17 @@ test('stream args validation', (t) => {
     path: '/',
     method: 'GET'
   }, null, (err) => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 
   client.stream(null, null, (err) => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 
   try {
     client.stream(null, null, 'asd')
   } catch (err) {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   }
 })
 
@@ -285,11 +285,11 @@ test('stream args validation promise', (t) => {
     path: '/',
     method: 'GET'
   }, null).catch((err) => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 
   client.stream(null, null).catch((err) => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 })
 
@@ -337,7 +337,7 @@ test('stream server side destroy', (t) => {
     }, () => {
       t.fail()
     }, (err) => {
-      t.type(err, errors.SocketError)
+      t.ok(err instanceof errors.SocketError)
     })
   })
 })
@@ -360,7 +360,7 @@ test('stream invalid return', (t) => {
     }, () => {
       return {}
     }, (err) => {
-      t.type(err, errors.InvalidReturnValueError)
+      t.ok(err instanceof errors.InvalidReturnValueError)
     })
   })
 })
@@ -413,7 +413,7 @@ test('stream factory abort', (t) => {
       return new PassThrough()
     }, (err) => {
       t.equal(signal.listenerCount('abort'), 0)
-      t.type(err, errors.RequestAbortedError)
+      t.ok(err instanceof errors.RequestAbortedError)
     })
     t.equal(signal.listenerCount('abort'), 1)
   })
@@ -475,7 +475,7 @@ test('stream CONNECT throw', (t) => {
       method: 'CONNECT'
     }, () => {
     }, (err) => {
-      t.type(err, errors.InvalidArgumentError)
+      t.ok(err instanceof errors.InvalidArgumentError)
     })
   })
 })
@@ -528,7 +528,7 @@ test('stream abort before dispatch', (t) => {
     }, () => {
       return pt
     }, (err) => {
-      t.type(err, errors.RequestAbortedError)
+      t.ok(err instanceof errors.RequestAbortedError)
     })
     signal.emit('abort')
   })

test/client-timeout.js

@@ -40,7 +40,7 @@ test('refresh timeout on pause', (t) => {
 
       },
       onError (err) {
-        t.type(err, errors.BodyTimeoutError)
+        t.ok(err instanceof errors.BodyTimeoutError)
       }
     })
   })
@@ -96,7 +96,7 @@ test('start headers timeout after request body', (t) => {
       },
       onError (err) {
         t.equal(body.readableEnded, true)
-        t.type(err, errors.HeadersTimeoutError)
+        t.ok(err instanceof errors.HeadersTimeoutError)
       }
     })
   })
@@ -153,7 +153,7 @@ test('start headers timeout after async iterator request body', (t) => {
 
       },
       onError (err) {
-        t.type(err, errors.HeadersTimeoutError)
+        t.ok(err instanceof errors.HeadersTimeoutError)
       }
     })
   })

test/client-upgrade.js

@@ -151,23 +151,23 @@ test('upgrade invalid opts', (t) => {
   const client = new Client('http://localhost:5432')
 
   client.upgrade(null, err => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
     t.equal(err.message, 'invalid opts')
   })
 
   try {
     client.upgrade(null, null)
     t.fail()
   } catch (err) {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
     t.equal(err.message, 'invalid opts')
   }
 
   try {
     client.upgrade({ path: '/' }, null)
     t.fail()
   } catch (err) {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
     t.equal(err.message, 'invalid callback')
   }
 })
@@ -308,7 +308,7 @@ test('upgrade aborted', (t) => {
       opaque: 'asd'
     }, (err, { opaque }) => {
       t.equal(opaque, 'asd')
-      t.type(err, errors.RequestAbortedError)
+      t.ok(err instanceof errors.RequestAbortedError)
       t.equal(signal.listenerCount('abort'), 0)
     })
     t.equal(client[kBusy], true)
@@ -351,7 +351,7 @@ test('basic aborted after res', (t) => {
       protocol: 'Websocket',
       signal
     }, (err) => {
-      t.type(err, errors.RequestAbortedError)
+      t.ok(err instanceof errors.RequestAbortedError)
     })
   })
 })
@@ -408,7 +408,7 @@ test('upgrade disconnect', (t) => {
 
     client.on('disconnect', (origin, [self], error) => {
       t.equal(client, self)
-      t.type(error, Error)
+      t.ok(error instanceof Error)
     })
 
     client
@@ -417,7 +417,7 @@ test('upgrade disconnect', (t) => {
         t.fail()
       })
       .catch(error => {
-        t.type(error, Error)
+        t.ok(error instanceof Error)
       })
   })
 })
@@ -446,7 +446,7 @@ test('upgrade invalid signal', (t) => {
       opaque: 'asd'
     }, (err, { opaque }) => {
       t.equal(opaque, 'asd')
-      t.type(err, errors.InvalidArgumentError)
+      t.ok(err instanceof errors.InvalidArgumentError)
     })
   })
 })

test/client.js

@@ -1412,13 +1412,13 @@ test('request args validation', (t) => {
   const client = new Client('http://localhost:5000')
 
   client.request(null, (err) => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 
   try {
     client.request(null, 'asd')
   } catch (err) {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   }
 })
 
@@ -1428,7 +1428,7 @@ test('request args validation promise', (t) => {
   const client = new Client('http://localhost:5000')
 
   client.request(null).catch((err) => {
-    t.type(err, errors.InvalidArgumentError)
+    t.ok(err instanceof errors.InvalidArgumentError)
   })
 })