chore: increase test coverage to 100% for /lib/api/api-request.js (#2912

)
nodejs · Mar 5, 2024 · 4852c23 · 4852c23
1 parent f4ca0be
commit 4852c23
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/test/node-test/client-errors.js b/test/node-test/client-errors.js
@@ -1056,7 +1056,7 @@ test('retry idempotent inflight', async (t) => {
 })
 
 test('invalid opts', async (t) => {
-  const p = tspl(t, { plan: 2 })
+  const p = tspl(t, { plan: 5 })
 
   const client = new Client('http://localhost:5000')
   client.request(null, (err) => {
@@ -1065,6 +1065,22 @@ test('invalid opts', async (t) => {
   client.pipeline(null).on('error', (err) => {
     p.ok(err instanceof errors.InvalidArgumentError)
   })
+  client.request({
+    path: '/',
+    method: 'GET',
+    highWaterMark: '1000'
+  }, (err) => {
+    p.ok(err instanceof errors.InvalidArgumentError)
+    p.strictEqual(err.message, 'invalid highWaterMark')
+  })
+  client.request({
+    path: '/',
+    method: 'GET',
+    highWaterMark: -1
+  }, (err) => {
+    p.ok(err instanceof errors.InvalidArgumentError)
+    p.strictEqual(err.message, 'invalid highWaterMark')
+  })
 
   await p.completed
 })