Commit 99e66097 authored by Markus Koller's avatar Markus Koller

Merge branch 'issue#220040-fix-rails-savebang-gitaly-client-module' into 'master'

Fix Rails/SaveBang Rubocop offenses for gitaly client models

See merge request gitlab-org/gitlab!58089
parents fef90e53 a9540088
...@@ -180,8 +180,6 @@ Rails/SaveBang: ...@@ -180,8 +180,6 @@ Rails/SaveBang:
- 'spec/lib/gitlab/email/handler/unsubscribe_handler_spec.rb' - 'spec/lib/gitlab/email/handler/unsubscribe_handler_spec.rb'
- 'spec/lib/gitlab/gfm/reference_rewriter_spec.rb' - 'spec/lib/gitlab/gfm/reference_rewriter_spec.rb'
- 'spec/lib/gitlab/git_access_spec.rb' - 'spec/lib/gitlab/git_access_spec.rb'
- 'spec/lib/gitlab/gitaly_client/object_pool_service_spec.rb'
- 'spec/lib/gitlab/gitaly_client/repository_service_spec.rb'
- 'spec/lib/gitlab/import_export/avatar_saver_spec.rb' - 'spec/lib/gitlab/import_export/avatar_saver_spec.rb'
- 'spec/lib/gitlab/import_export/base/relation_factory_spec.rb' - 'spec/lib/gitlab/import_export/base/relation_factory_spec.rb'
- 'spec/lib/gitlab/import_export/design_repo_restorer_spec.rb' - 'spec/lib/gitlab/import_export/design_repo_restorer_spec.rb'
......
---
title: Fix Rails/SaveBang Rubocop offenses for gitaly client models
merge_request: 58089
author: Huzaifa Iftikhar @huzaifaiftikhar
type: fixed
...@@ -11,7 +11,7 @@ RSpec.describe Gitlab::GitalyClient::ObjectPoolService do ...@@ -11,7 +11,7 @@ RSpec.describe Gitlab::GitalyClient::ObjectPoolService do
subject { described_class.new(object_pool) } subject { described_class.new(object_pool) }
before do before do
subject.create(raw_repository) subject.create(raw_repository) # rubocop:disable Rails/SaveBang
end end
describe '#create' do describe '#create' do
...@@ -22,7 +22,7 @@ RSpec.describe Gitlab::GitalyClient::ObjectPoolService do ...@@ -22,7 +22,7 @@ RSpec.describe Gitlab::GitalyClient::ObjectPoolService do
context 'when the pool already exists' do context 'when the pool already exists' do
it 'returns an error' do it 'returns an error' do
expect do expect do
subject.create(raw_repository) subject.create(raw_repository) # rubocop:disable Rails/SaveBang
end.to raise_error(GRPC::FailedPrecondition) end.to raise_error(GRPC::FailedPrecondition)
end end
end end
......
...@@ -270,7 +270,7 @@ RSpec.describe Gitlab::GitalyClient::RepositoryService do ...@@ -270,7 +270,7 @@ RSpec.describe Gitlab::GitalyClient::RepositoryService do
let(:object_pool_service) { Gitlab::GitalyClient::ObjectPoolService.new(object_pool) } let(:object_pool_service) { Gitlab::GitalyClient::ObjectPoolService.new(object_pool) }
before do before do
object_pool_service.create(repository) object_pool_service.create(repository) # rubocop:disable Rails/SaveBang
object_pool_service.link_repository(repository) object_pool_service.link_repository(repository)
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment