Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
473ccd7f
Commit
473ccd7f
authored
Apr 07, 2021
by
Abdul Wadood
Committed by
Rémy Coutable
Apr 07, 2021
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Fix Rails/SaveBang offenses in spec/initializers
parent
143bbd46
Changes
4
Hide whitespace changes
Inline
Side-by-side
Showing
4 changed files
with
9 additions
and
4 deletions
+9
-4
.rubocop_manual_todo.yml
.rubocop_manual_todo.yml
+0
-2
changelogs/unreleased/rails-save-bang-initializers.yml
changelogs/unreleased/rails-save-bang-initializers.yml
+5
-0
spec/initializers/active_record_locking_spec.rb
spec/initializers/active_record_locking_spec.rb
+2
-2
spec/initializers/fog_google_https_private_urls_spec.rb
spec/initializers/fog_google_https_private_urls_spec.rb
+2
-0
No files found.
.rubocop_manual_todo.yml
View file @
473ccd7f
...
...
@@ -203,8 +203,6 @@ Rails/SaveBang:
-
'
spec/graphql/mutations/merge_requests/set_locked_spec.rb'
-
'
spec/graphql/mutations/merge_requests/set_wip_spec.rb'
-
'
spec/graphql/resolvers/boards_resolver_spec.rb'
-
'
spec/initializers/active_record_locking_spec.rb'
-
'
spec/initializers/fog_google_https_private_urls_spec.rb'
-
'
spec/lib/after_commit_queue_spec.rb'
-
'
spec/lib/backup/manager_spec.rb'
-
'
spec/lib/gitlab/alerting/alert_spec.rb'
...
...
changelogs/unreleased/rails-save-bang-initializers.yml
0 → 100644
View file @
473ccd7f
---
title
:
Fix Rails/SaveBang rubocop offenses in spec/initializers
merge_request
:
58049
author
:
Abdul Wadood @abdulwd
type
:
fixed
spec/initializers/active_record_locking_spec.rb
View file @
473ccd7f
...
...
@@ -11,13 +11,13 @@ RSpec.describe 'ActiveRecord locking' do
end
it
'can be updated'
do
issue
.
update
(
title:
"New title"
)
issue
.
update
!
(
title:
"New title"
)
expect
(
issue
.
reload
.
lock_version
).
to
eq
(
new_lock_version
)
end
it
'can be deleted'
do
expect
{
issue
.
destroy
}.
to
change
{
Issue
.
count
}.
by
(
-
1
)
expect
{
issue
.
destroy
!
}.
to
change
{
Issue
.
count
}.
by
(
-
1
)
end
end
...
...
spec/initializers/fog_google_https_private_urls_spec.rb
View file @
473ccd7f
...
...
@@ -13,11 +13,13 @@ RSpec.describe 'Fog::Storage::GoogleXML::File', :fog_requests do
end
let
(
:file
)
do
# rubocop:disable Rails/SaveBang
directory
=
storage
.
directories
.
create
(
key:
'data'
)
directory
.
files
.
create
(
body:
'Hello World!'
,
key:
'hello_world.txt'
)
# rubocop:enable Rails/SaveBang
end
it
'delegates to #get_https_url'
do
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment