Use .includes instead of .indexOf

victorb · victorb · commit 4eb223ecc19a · 2016-09-14T13:18:31.000+02:00
diff --git a/src/api/pubsub.js b/src/api/pubsub.js
@@ -10,7 +10,7 @@ const http = require('http')
 let activeSubscriptions = []
 
 const subscriptionExists = (subscriptions, topic) => {
-  return subscriptions.indexOf(topic) !== -1
+  return subscriptions.includes(topic)
 }
 const removeSubscription = (subscriptions, topic) => {
   const indexToRemove = subscriptions.indexOf(topic)

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ const http = require('http')`
`10`	`10`	`let activeSubscriptions = []`
`11`	`11`
`12`	`12`	`const subscriptionExists = (subscriptions, topic) => {`
`13`		`- return subscriptions.indexOf(topic) !== -1`
	`13`	`+ return subscriptions.includes(topic)`
`14`	`14`	`}`
`15`	`15`	`const removeSubscription = (subscriptions, topic) => {`
`16`	`16`	`const indexToRemove = subscriptions.indexOf(topic)`