Lint Style/RedundantBegin

.rubocop_todo.yml

@@ -67,9 +67,3 @@ Performance/StringInclude:
   Exclude:
     - 'spec/support/fake_safe_buffer.rb'
 
-# Offense count: 2
-# Cop supports --auto-correct.
-Style/RedundantBegin:
-  Exclude:
-    - 'spec/faraday/request/instrumentation_spec.rb'
-    - 'spec/faraday/response/json_spec.rb'

spec/faraday/request/instrumentation_spec.rb

@@ -30,13 +30,11 @@ def instrument(name, env)
 
   it { expect(options.name).to eq('request.faraday') }
   it 'defaults to ActiveSupport::Notifications' do
-    begin
-      res = options.instrumenter
-    rescue NameError => e
-      expect(e.to_s).to match('ActiveSupport')
-    else
-      expect(res).to eq(ActiveSupport::Notifications)
-    end
+    res = options.instrumenter
+  rescue NameError => e
+    expect(e.to_s).to match('ActiveSupport')
+  else
+    expect(res).to eq(ActiveSupport::Notifications)
   end
 
   it 'instruments with default name' do

spec/faraday/response/json_spec.rb

@@ -76,12 +76,10 @@ def process(body, content_type = 'application/json', options = {})
   end
 
   it 'includes the response on the ParsingError instance' do
-    begin
-      process('{') { |env| env[:response] = Faraday::Response.new }
-      raise 'Parsing should have failed.'
-    rescue Faraday::ParsingError => e
-      expect(e.response).to be_a(Faraday::Response)
-    end
+    process('{') { |env| env[:response] = Faraday::Response.new }
+    raise 'Parsing should have failed.'
+  rescue Faraday::ParsingError => e
+    expect(e.response).to be_a(Faraday::Response)
   end
 
   context 'HEAD responses' do