fixed Style/RaiseArgs

Author: SHinGo-Koba

spec/rspec/rails/matchers/be_routable_spec.rb

@@ -17,7 +17,7 @@
     end
 
     it "fails if routes do not recognize the path" do
-      allow(routes).to receive(:recognize_path) { raise ActionController::RoutingError.new('ignore') }
+      allow(routes).to receive(:recognize_path) { raise ActionController::RoutingError, 'ignore' }
       expect do
         expect({ get: "/a/path" }).to be_routable
       end.to raise_error(/expected \{:get=>"\/a\/path"\} to be routable/)
@@ -27,7 +27,7 @@
   context "with should_not" do
 
     it "passes if routes do not recognize the path" do
-      allow(routes).to receive(:recognize_path) { raise ActionController::RoutingError.new('ignore') }
+      allow(routes).to receive(:recognize_path) { raise ActionController::RoutingError, 'ignore' }
       expect do
         expect({ get: "/a/path" }).not_to be_routable
       end.to_not raise_error

spec/rspec/rails/matchers/have_rendered_spec.rb

@@ -40,7 +40,7 @@ def assert_template(*); end
       context "when assert_template fails" do
         it "uses failure message from assert_template" do
           def assert_template(*)
-            raise ActiveSupport::TestCase::Assertion.new("this message")
+            raise ActiveSupport::TestCase::Assertion, "this message"
           end
           expect do
             expect(response).to send(template_expectation, "template_name")
@@ -64,7 +64,7 @@ def assert_template(*)
       context "when assert_template fails" do
         it "passes" do
           def assert_template(*)
-            raise ActiveSupport::TestCase::Assertion.new("this message")
+            raise ActiveSupport::TestCase::Assertion, "this message"
           end
           expect do
             expect(response).to_not send(template_expectation, "template_name")
@@ -77,7 +77,7 @@ def assert_template(*)
           def assert_template(*); end
           expect do
             expect(response).to_not send(template_expectation, "template_name")
-          end.to raise_error(/expected not to render \"template_name\", but did/)
+          end.to raise_error(/expected not to render "template_name", but did/)
         end
       end
 
@@ -95,7 +95,7 @@ def assert_template(*); raise "oops"; end
 
         def assert_template(*)
           message = "expecting <'template_name'> but rendering with <[]>"
-          raise ActiveSupport::TestCase::Assertion.new(message)
+          raise ActiveSupport::TestCase::Assertion, message
         end
 
         def normalize_argument_to_redirection(_response_redirect_location)
@@ -113,7 +113,7 @@ def normalize_argument_to_redirection(_response_redirect_location)
         context 'with a badly formatted error message' do
           def assert_template(*)
             message = 'expected [] to include "some/path"'
-            raise ActiveSupport::TestCase::Assertion.new(message)
+            raise ActiveSupport::TestCase::Assertion, message
           end
 
           it 'falls back to something informative' do

spec/rspec/rails/matchers/redirect_to_spec.rb

@@ -19,7 +19,7 @@ def assert_redirected_to(*); end
 
     context "when assert_redirected_to fails" do
       def assert_redirected_to(*)
-        raise ActiveSupport::TestCase::Assertion.new("this message")
+        raise ActiveSupport::TestCase::Assertion, "this message"
       end
 
       it "uses failure message from assert_redirected_to" do
@@ -45,7 +45,7 @@ def assert_redirected_to(*)
   context "with should_not" do
     context "when assert_redirected_to fails" do
       def assert_redirected_to(*)
-        raise ActiveSupport::TestCase::Assertion.new("this message")
+        raise ActiveSupport::TestCase::Assertion, "this message"
       end
 
       it "passes" do
@@ -61,7 +61,7 @@ def assert_redirected_to(*); end
       it "fails with custom failure message" do
         expect do
           expect(response).not_to redirect_to("destination")
-        end.to raise_exception(/expected not to redirect to \"destination\", but did/)
+        end.to raise_exception(/expected not to redirect to "destination", but did/)
       end
     end
 

spec/rspec/rails/matchers/route_to_spec.rb

@@ -73,7 +73,7 @@ def assert_recognizes(*)
     context "when assert_recognizes fails with an assertion failure" do
       it "fails with message from assert_recognizes" do
         def assert_recognizes(*)
-          raise ActiveSupport::TestCase::Assertion.new("this message")
+          raise ActiveSupport::TestCase::Assertion, "this message"
         end
         expect do
           expect({ get: "path" }).to route_to("these" => "options")
@@ -84,7 +84,7 @@ def assert_recognizes(*)
     context "when assert_recognizes fails with a routing error" do
       it "fails with message from assert_recognizes" do
         def assert_recognizes(*)
-          raise ActionController::RoutingError.new("this message")
+          raise ActionController::RoutingError, "this message"
         end
         expect do
           expect({ get: "path" }).to route_to("these" => "options")
@@ -116,7 +116,7 @@ def assert_recognizes(*)
     context "when assert_recognizes fails with an assertion failure" do
       it "passes" do
         def assert_recognizes(*)
-          raise ActiveSupport::TestCase::Assertion.new("this message")
+          raise ActiveSupport::TestCase::Assertion, "this message"
         end
         expect do
           expect({ get: "path" }).not_to route_to("these" => "options")
@@ -127,7 +127,7 @@ def assert_recognizes(*)
     context "when assert_recognizes fails with a routing error" do
       it "passes" do
         def assert_recognizes(*)
-          raise ActionController::RoutingError.new("this message")
+          raise ActionController::RoutingError, "this message"
         end
         expect do
           expect({ get: "path" }).not_to route_to("these" => "options")