Commit 312b43feb4863328109fe5a5d7aea642157a8e1a
1 parent
f2c11950
Exists in
master
and in
1 other branch
Rubocop: remove unnecessary method call parentheses
Showing
3 changed files
with
6 additions
and
13 deletions
Show diff stats
.rubocop_todo.yml
| @@ -341,13 +341,6 @@ Style/LeadingCommentSpace: | @@ -341,13 +341,6 @@ Style/LeadingCommentSpace: | ||
| 341 | - 'spec/models/notification_service/hoiio_service_spec.rb' | 341 | - 'spec/models/notification_service/hoiio_service_spec.rb' |
| 342 | - 'spec/models/notification_service/pushover_service_spec.rb' | 342 | - 'spec/models/notification_service/pushover_service_spec.rb' |
| 343 | 343 | ||
| 344 | -# Offense count: 6 | ||
| 345 | -# Cop supports --auto-correct. | ||
| 346 | -Style/MethodCallParentheses: | ||
| 347 | - Exclude: | ||
| 348 | - - 'spec/controllers/api/v1/notices_controller_spec.rb' | ||
| 349 | - - 'spec/controllers/api/v1/problems_controller_spec.rb' | ||
| 350 | - | ||
| 351 | # Offense count: 3 | 344 | # Offense count: 3 |
| 352 | # Cop supports --auto-correct. | 345 | # Cop supports --auto-correct. |
| 353 | # Configuration parameters: EnforcedStyle, SupportedStyles. | 346 | # Configuration parameters: EnforcedStyle, SupportedStyles. |
spec/controllers/api/v1/notices_controller_spec.rb
| @@ -14,7 +14,7 @@ describe Api::V1::NoticesController, type: 'controller' do | @@ -14,7 +14,7 @@ describe Api::V1::NoticesController, type: 'controller' do | ||
| 14 | 14 | ||
| 15 | it "should return JSON if JSON is requested" do | 15 | it "should return JSON if JSON is requested" do |
| 16 | get :index, :auth_token => @user.authentication_token, :format => "json" | 16 | get :index, :auth_token => @user.authentication_token, :format => "json" |
| 17 | - expect { JSON.load(response.body) }.not_to raise_error() #JSON::ParserError) | 17 | + expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError) |
| 18 | end | 18 | end |
| 19 | 19 | ||
| 20 | it "should return XML if XML is requested" do | 20 | it "should return XML if XML is requested" do |
| @@ -24,7 +24,7 @@ describe Api::V1::NoticesController, type: 'controller' do | @@ -24,7 +24,7 @@ describe Api::V1::NoticesController, type: 'controller' do | ||
| 24 | 24 | ||
| 25 | it "should return JSON by default" do | 25 | it "should return JSON by default" do |
| 26 | get :index, :auth_token => @user.authentication_token | 26 | get :index, :auth_token => @user.authentication_token |
| 27 | - expect { JSON.load(response.body) }.not_to raise_error() #JSON::ParserError) | 27 | + expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError) |
| 28 | end | 28 | end |
| 29 | 29 | ||
| 30 | describe "given a date range" do | 30 | describe "given a date range" do |
spec/controllers/api/v1/problems_controller_spec.rb
| @@ -13,7 +13,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | @@ -13,7 +13,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | ||
| 13 | 13 | ||
| 14 | it "should return JSON if JSON is requested" do | 14 | it "should return JSON if JSON is requested" do |
| 15 | get :show, :auth_token => @user.authentication_token, :format => "json", :id => Problem.first.id | 15 | get :show, :auth_token => @user.authentication_token, :format => "json", :id => Problem.first.id |
| 16 | - expect { JSON.load(response.body) }.not_to raise_error() #JSON::ParserError | 16 | + expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError |
| 17 | end | 17 | end |
| 18 | 18 | ||
| 19 | it "should return XML if XML is requested" do | 19 | it "should return XML if XML is requested" do |
| @@ -23,7 +23,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | @@ -23,7 +23,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | ||
| 23 | 23 | ||
| 24 | it "should return JSON by default" do | 24 | it "should return JSON by default" do |
| 25 | get :show, :auth_token => @user.authentication_token, :id => @problem.id | 25 | get :show, :auth_token => @user.authentication_token, :id => @problem.id |
| 26 | - expect { JSON.load(response.body) }.not_to raise_error()#JSON::ParserError) | 26 | + expect { JSON.load(response.body) }.not_to raise_error#JSON::ParserError) |
| 27 | end | 27 | end |
| 28 | 28 | ||
| 29 | it "should return the correct problem" do | 29 | it "should return the correct problem" do |
| @@ -68,7 +68,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | @@ -68,7 +68,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | ||
| 68 | 68 | ||
| 69 | it "should return JSON if JSON is requested" do | 69 | it "should return JSON if JSON is requested" do |
| 70 | get :index, :auth_token => @user.authentication_token, :format => "json" | 70 | get :index, :auth_token => @user.authentication_token, :format => "json" |
| 71 | - expect { JSON.load(response.body) }.not_to raise_error()#JSON::ParserError) | 71 | + expect { JSON.load(response.body) }.not_to raise_error#JSON::ParserError) |
| 72 | end | 72 | end |
| 73 | 73 | ||
| 74 | it "should return XML if XML is requested" do | 74 | it "should return XML if XML is requested" do |
| @@ -78,7 +78,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | @@ -78,7 +78,7 @@ describe Api::V1::ProblemsController, type: 'controller' do | ||
| 78 | 78 | ||
| 79 | it "should return JSON by default" do | 79 | it "should return JSON by default" do |
| 80 | get :index, :auth_token => @user.authentication_token | 80 | get :index, :auth_token => @user.authentication_token |
| 81 | - expect { JSON.load(response.body) }.not_to raise_error()#JSON::ParserError) | 81 | + expect { JSON.load(response.body) }.not_to raise_error#JSON::ParserError) |
| 82 | end | 82 | end |
| 83 | 83 | ||
| 84 | describe "given a date range" do | 84 | describe "given a date range" do |