Merge pull request #778 from ipfs/fix/ping-pong

alanshaw · web-flow · commit 3c695af98bbd · 2018-05-29T12:16:51.000+01:00
fix: correctly differentiate pong responses
diff --git a/test/ping.spec.js b/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 () {

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ const f = require('./utils/factory')`
`17`	`17`
`18`	`18`	`// Determine if a ping response object is a pong, or something else, like a status message`
`19`	`19`	`function isPong (pingResponse) {`
`20`		`- return Boolean(pingResponse && pingResponse.time)`
	`20`	`+ return Boolean(pingResponse && pingResponse.success && !pingResponse.text)`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`describe('.ping', function () {`