[Refactor] Suppress warnings from Ruby

lib/generators/ember/bootstrap_generator.rb

@@ -21,7 +21,7 @@ def inject_ember
           else
             inject_into_application_file(engine_extension)
           end
-        rescue Exception => e
+        rescue Exception
           inject_into_application_file('js')
         end
       end
@@ -69,14 +69,14 @@ def inject_into_application_file(safe_extension)
       end
 
       def get_options_from_contents(contents)
-        if contents =~ regex = /^.*require_tree.*$/
-                                {:before => regex}
-                              elsif contents =~ regex = /^\s*$/
-                                {:before => regex}
-                              else
-                                regex = /\z/
-                                {:after => regex}
-                              end
+        regex = /^.*require_tree.*$/
+        return {:before => regex} if contents =~ regex
+
+        regex = /^\s*$/
+        return {:before => regex} if contents =~ regex
+
+        regex = /\z/
+        {:after => regex}
       end
     end
   end

lib/generators/ember/install_generator.rb

@@ -181,7 +181,7 @@ def fetch(from, to, prepend_verbose = true)
           raise Thor::Error
         end
         output.rewind
-        content = output.read
+        output.read
       end
 
       def resource_exist?(target)

test/hjstemplate_test.rb

@@ -15,26 +15,26 @@ class HjsTemplateTest < IntegrationTest
   test "asset pipeline should serve template" do
     get "/assets/templates/test.js"
     assert_response :success
-    assert_match /Ember\.TEMPLATES\["test"\] = Ember\.(?:Handlebars|HTMLBars)\.template\(/m, @response.body
+    assert_match %r{Ember\.TEMPLATES\["test"\] = Ember\.(?:Handlebars|HTMLBars)\.template\(}m, @response.body
   end
 
   test "asset pipeline should serve bundled application.js" do
     get "/assets/application.js"
     assert_response :success
-    assert_match /Ember\.TEMPLATES\["test"\] = Ember\.(?:Handlebars|HTMLBars)\.template\(/m, @response.body
+    assert_match %r{Ember\.TEMPLATES\["test"\] = Ember\.(?:Handlebars|HTMLBars)\.template\(}m, @response.body
   end
 
   test "should unbind mustache templates" do
     get "/assets/templates/hairy.js"
     assert_response :success
-    assert_match /Ember\.TEMPLATES\["hairy(\.mustache)?"\] = Ember\.(?:Handlebars|HTMLBars)\.template\(/m, @response.body
-    assert_match /function .*unbound|"name":"unbound"/m, @response.body
+    assert_match %r{Ember\.TEMPLATES\["hairy(\.mustache)?"\] = Ember\.(?:Handlebars|HTMLBars)\.template\(}m, @response.body
+    assert_match %r{function .*unbound|"name":"unbound"}m, @response.body
   end
 
   test "ensure new lines inside the anon function are persisted" do
     get "/assets/templates/new_lines.js"
     assert_response :success
-    assert @response.body =~ /; data = data || {};\n|"data":data/, @response.body.inspect
+    assert @response.body =~ %r(; data = data || {};\n|"data":data), @response.body.inspect
   end
 
 end

test/precompile_test.rb

@@ -28,8 +28,8 @@ def precompile!(rails_env)
 
     contents = File.read(application_js_path)
 
-    assert_match /Ember\.VERSION/, contents, 'application.js should contain Ember.VERSION'
-    assert_match /Handlebars\.VERSION|COMPILER_REVISION/, contents, 'applciation.js should contain Handlebars.VERSION'
+    assert_match %r{Ember\.VERSION}, contents, 'application.js should contain Ember.VERSION'
+    assert_match %r{Handlebars\.VERSION|COMPILER_REVISION}, contents, 'applciation.js should contain Handlebars.VERSION'
   end
 
   def app_path