Split, rather than slicing. Also use a variable for clarity.

technicalpickles · web-flow · commit 2176f8a792a3 · 2017-03-15T09:56:34.000-04:00
diff --git a/lib/chatops/controller/test_case_helpers.rb b/lib/chatops/controller/test_case_helpers.rb
@@ -17,7 +17,8 @@ def chatop(method, params = {})
       :user => user
     }
 
-    if Rails.version.slice(0).to_i >= 5
+    major_version = Rails.version.split('.')[0].to_i
+    if major_version >= 5
       post :execute_chatop, params: params.merge(chatop: method)
     else
       post :execute_chatop, params.merge(chatop: method)

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,8 @@ def chatop(method, params = {})`
`17`	`17`	`:user => user`
`18`	`18`	`}`
`19`	`19`
`20`		`- if Rails.version.slice(0).to_i >= 5`
	`20`	`+ major_version = Rails.version.split('.')[0].to_i`
	`21`	`+ if major_version >= 5`
`21`	`22`	`post :execute_chatop, params: params.merge(chatop: method)`
`22`	`23`	`else`
`23`	`24`	`post :execute_chatop, params.merge(chatop: method)`