fix: correctly differentiate pong responses

Investigation discovered that pong responses CAN have `time: 0` (they can be very quick). Previously pong messages were differentiated by time greater than 0, but considering it can be 0 this was incorrect.

License: MIT
Signed-off-by: Alan Shaw <alan@tableflip.io>

test/ping.spec.js

@@ -17,7 +17,7 @@ const f = require('./utils/factory')
 
 // Determine if a ping response object is a pong, or something else, like a status message
 function isPong (pingResponse) {
-  return Boolean(pingResponse && pingResponse.time)
+  return Boolean(pingResponse && pingResponse.success && !pingResponse.text)
 }
 
 describe('.ping', function () {