Commit af47f58dfadb04cc48e25ed807cecac195a56c78
1 parent
ac0691c0
Exists in
master
and in
1 other branch
Rubocop: fix lint warnings: space before arguments to method call
Fixes: $ rubocop --lint --only Lint/ParenthesesAsGroupedExpression
Showing
4 changed files
with
8 additions
and
8 deletions
Show diff stats
spec/controllers/users_controller_spec.rb
| @@ -31,7 +31,7 @@ describe UsersController, type: 'controller' do | @@ -31,7 +31,7 @@ describe UsersController, type: 'controller' do | ||
| 31 | context "GET /users/:my_id/edit" do | 31 | context "GET /users/:my_id/edit" do |
| 32 | it 'finds the user' do | 32 | it 'finds the user' do |
| 33 | get :edit, :id => user.id | 33 | get :edit, :id => user.id |
| 34 | - expect(controller.user).to eq user | 34 | + expect(controller.user).to eq(user) |
| 35 | expect(response).to render_template 'edit' | 35 | expect(response).to render_template 'edit' |
| 36 | end | 36 | end |
| 37 | 37 | ||
| @@ -234,12 +234,12 @@ describe UsersController, type: 'controller' do | @@ -234,12 +234,12 @@ describe UsersController, type: 'controller' do | ||
| 234 | } | 234 | } |
| 235 | let(:user_param) { {'user' => { :name => 'foo', :admin => true }} } | 235 | let(:user_param) { {'user' => { :name => 'foo', :admin => true }} } |
| 236 | it 'not have admin field' do | 236 | it 'not have admin field' do |
| 237 | - expect(controller.send(:user_params)).to eq ({'name' => 'foo'}) | 237 | + expect(controller.send(:user_params)).to eq({'name' => 'foo'}) |
| 238 | end | 238 | end |
| 239 | context "with password and password_confirmation empty?" do | 239 | context "with password and password_confirmation empty?" do |
| 240 | let(:user_param) { {'user' => { :name => 'foo', 'password' => '', 'password_confirmation' => '' }} } | 240 | let(:user_param) { {'user' => { :name => 'foo', 'password' => '', 'password_confirmation' => '' }} } |
| 241 | it 'not have password and password_confirmation field' do | 241 | it 'not have password and password_confirmation field' do |
| 242 | - expect(controller.send(:user_params)).to eq ({'name' => 'foo'}) | 242 | + expect(controller.send(:user_params)).to eq({'name' => 'foo'}) |
| 243 | end | 243 | end |
| 244 | end | 244 | end |
| 245 | end | 245 | end |
spec/interactors/problem_merge_spec.rb
| @@ -36,7 +36,7 @@ describe ProblemMerge do | @@ -36,7 +36,7 @@ describe ProblemMerge do | ||
| 36 | 36 | ||
| 37 | it 'move all err in one problem' do | 37 | it 'move all err in one problem' do |
| 38 | problem_merge.merge | 38 | problem_merge.merge |
| 39 | - expect(problem.reload.errs.map(&:id).sort).to eq (first_errs | merged_errs).map(&:id).sort | 39 | + expect(problem.reload.errs.map(&:id).sort).to eq((first_errs | merged_errs).map(&:id).sort) |
| 40 | end | 40 | end |
| 41 | 41 | ||
| 42 | it 'keeps the issue link' do | 42 | it 'keeps the issue link' do |
spec/models/notice_spec.rb
| @@ -107,7 +107,7 @@ describe Notice, type: 'model' do | @@ -107,7 +107,7 @@ describe Notice, type: 'model' do | ||
| 107 | describe "request" do | 107 | describe "request" do |
| 108 | it "returns empty hash if not set" do | 108 | it "returns empty hash if not set" do |
| 109 | notice = Notice.new | 109 | notice = Notice.new |
| 110 | - expect(notice.request).to eq ({}) | 110 | + expect(notice.request).to eq({}) |
| 111 | end | 111 | end |
| 112 | end | 112 | end |
| 113 | 113 |
spec/models/problem_spec.rb
| @@ -263,7 +263,7 @@ describe Problem, type: 'model' do | @@ -263,7 +263,7 @@ describe Problem, type: 'model' do | ||
| 263 | end | 263 | end |
| 264 | 264 | ||
| 265 | it "#messages should be empty by default" do | 265 | it "#messages should be empty by default" do |
| 266 | - expect(@problem.messages).to eq ({}) | 266 | + expect(@problem.messages).to eq({}) |
| 267 | end | 267 | end |
| 268 | 268 | ||
| 269 | it "removing a notice removes string from #messages" do | 269 | it "removing a notice removes string from #messages" do |
| @@ -290,7 +290,7 @@ describe Problem, type: 'model' do | @@ -290,7 +290,7 @@ describe Problem, type: 'model' do | ||
| 290 | end | 290 | end |
| 291 | 291 | ||
| 292 | it "#hosts should be empty by default" do | 292 | it "#hosts should be empty by default" do |
| 293 | - expect(@problem.hosts).to eq ({}) | 293 | + expect(@problem.hosts).to eq({}) |
| 294 | end | 294 | end |
| 295 | 295 | ||
| 296 | it "removing a notice removes string from #hosts" do | 296 | it "removing a notice removes string from #hosts" do |
| @@ -310,7 +310,7 @@ describe Problem, type: 'model' do | @@ -310,7 +310,7 @@ describe Problem, type: 'model' do | ||
| 310 | end | 310 | end |
| 311 | 311 | ||
| 312 | it "#user_agents should be empty by default" do | 312 | it "#user_agents should be empty by default" do |
| 313 | - expect(@problem.user_agents).to eq ({}) | 313 | + expect(@problem.user_agents).to eq({}) |
| 314 | end | 314 | end |
| 315 | 315 | ||
| 316 | it "removing a notice removes string from #user_agents" do | 316 | it "removing a notice removes string from #user_agents" do |