test: http2 client ping errors

PR-URL: #18849
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: trivikr

test/parallel/test-http2-ping.js

@@ -71,12 +71,60 @@ server.listen(0, common.mustCall(() => {
 assert.deepStrictEqual(payload,ret);
 })));
 }
+
 // Only max 2 pings at a time based on the maxOutstandingPings option
 assert(!client.ping(common.expectsError({
 code: 'ERR_HTTP2_PING_CANCEL',
 type: Error,
 message: 'HTTP2 ping cancelled'
 })));
+
+// should throw if payload is not of type ArrayBufferView
+{
+[1,true,{},[]].forEach((invalidPayload)=>
+common.expectsError(
+()=>client.ping(invalidPayload),
+{
+type: TypeError,
+code: 'ERR_INVALID_ARG_TYPE',
+message: 'The "payload" argument must be one of type'+
+' Buffer, TypedArray, or DataView'
+}
+)
+);
+}
+
+// should throw if payload length is not 8
+{
+constshortPayload=Buffer.from('abcdefg');
+constlongPayload=Buffer.from('abcdefghi');
+[shortPayload,longPayload].forEach((payloadWithInvalidLength)=>
+common.expectsError(
+()=>client.ping(payloadWithInvalidLength),
+{
+type: RangeError,
+code: 'ERR_HTTP2_PING_LENGTH',
+message: 'HTTP2 ping payload must be 8 bytes'
+}
+)
+);
+}
+
+// should throw error is callback is not of type function
+{
+constpayload=Buffer.from('abcdefgh');
+[1,true,{},[]].forEach((invalidCallback)=>
+common.expectsError(
+()=>client.ping(payload,invalidCallback),
+{
+type: TypeError,
+code: 'ERR_INVALID_CALLBACK',
+message: 'callback must be a function'
+}
+)
+);
+}
+
 constreq=client.request();
 req.resume();
 req.on('end',common.mustCall(()=>{