From 06a74bf855ae20d3158644bc2745359fd97ae0ef Mon Sep 17 00:00:00 2001
From: Jeff Kreeftmeijer <jeff@kreeftmeijer.nl>
Date: Wed, 16 Jun 2010 15:25:21 +0200
Subject: [PATCH] got rid of the "ambiguous first argument; put parentheses or even spaces" warning in the scaffold_generator tests [#4872 state:resolved]

---
 .../test/generators/scaffold_generator_test.rb     |   30 ++++++++++----------
 1 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/railties/test/generators/scaffold_generator_test.rb b/railties/test/generators/scaffold_generator_test.rb
index e8e622f..20b1cbd 100644
--- a/railties/test/generators/scaffold_generator_test.rb
+++ b/railties/test/generators/scaffold_generator_test.rb
@@ -18,44 +18,44 @@ class ScaffoldGeneratorTest < Rails::Generators::TestCase
 
     # Route
     assert_file "config/routes.rb" do |route|
-      assert_match /resources :product_lines$/, route
+      assert_match(/resources :product_lines$/, route)
     end
 
     # Controller
     assert_file "app/controllers/product_lines_controller.rb" do |content|
-      assert_match /class ProductLinesController < ApplicationController/, content
+      assert_match(/class ProductLinesController < ApplicationController/, content)
 
       assert_instance_method :index, content do |m|
-        assert_match /@product_lines = ProductLine\.all/, m
+        assert_match(/@product_lines = ProductLine\.all/, m)
       end
 
       assert_instance_method :show, content do |m|
-        assert_match /@product_line = ProductLine\.find\(params\[:id\]\)/, m
+        assert_match(/@product_line = ProductLine\.find\(params\[:id\]\)/, m)
       end
 
       assert_instance_method :new, content do |m|
-        assert_match /@product_line = ProductLine\.new/, m
+        assert_match(/@product_line = ProductLine\.new/, m)
       end
 
       assert_instance_method :edit, content do |m|
-        assert_match /@product_line = ProductLine\.find\(params\[:id\]\)/, m
+        assert_match(/@product_line = ProductLine\.find\(params\[:id\]\)/, m)
       end
 
       assert_instance_method :create, content do |m|
-        assert_match /@product_line = ProductLine\.new\(params\[:product_line\]\)/, m
-        assert_match /@product_line\.save/, m
-        assert_match /@product_line\.errors/, m
+        assert_match(/@product_line = ProductLine\.new\(params\[:product_line\]\)/, m)
+        assert_match(/@product_line\.save/, m)
+        assert_match(/@product_line\.errors/, m)
       end
 
       assert_instance_method :update, content do |m|
-        assert_match /@product_line = ProductLine\.find\(params\[:id\]\)/, m
-        assert_match /@product_line\.update_attributes\(params\[:product_line\]\)/, m
-        assert_match /@product_line\.errors/, m
+        assert_match(/@product_line = ProductLine\.find\(params\[:id\]\)/, m)
+        assert_match(/@product_line\.update_attributes\(params\[:product_line\]\)/, m)
+        assert_match(/@product_line\.errors/, m)
       end
 
       assert_instance_method :destroy, content do |m|
-        assert_match /@product_line = ProductLine\.find\(params\[:id\]\)/, m
-        assert_match /@product_line\.destroy/, m
+        assert_match(/@product_line = ProductLine\.find\(params\[:id\]\)/, m)
+        assert_match(/@product_line\.destroy/, m)
       end
     end
 
@@ -92,7 +92,7 @@ class ScaffoldGeneratorTest < Rails::Generators::TestCase
 
     # Route
     assert_file "config/routes.rb" do |route|
-      assert_no_match /resources :product_lines$/, route
+      assert_no_match(/resources :product_lines$/, route)
     end
 
     # Controller
-- 
1.6.4.1