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
a0dd86b9
Commit
a0dd86b9
authored
Sep 03, 2020
by
Rajendra Kadam
Committed by
Rémy Coutable
Sep 03, 2020
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Fix Rails/SaveBang offenses for spec/requests/api/*
Fixes Rails/SaveBang cop for spec files
parent
b7539aa1
Changes
5
Show whitespace changes
Inline
Side-by-side
Showing
5 changed files
with
10 additions
and
8 deletions
+10
-8
.rubocop_todo.yml
.rubocop_todo.yml
+0
-3
changelogs/unreleased/rails-save-bang-28.yml
changelogs/unreleased/rails-save-bang-28.yml
+5
-0
spec/requests/api/access_requests_spec.rb
spec/requests/api/access_requests_spec.rb
+1
-1
spec/requests/api/boards_spec.rb
spec/requests/api/boards_spec.rb
+2
-2
spec/requests/api/branches_spec.rb
spec/requests/api/branches_spec.rb
+2
-2
No files found.
.rubocop_todo.yml
View file @
a0dd86b9
...
...
@@ -1172,9 +1172,6 @@ Rails/SaveBang:
-
'
spec/presenters/ci/build_runner_presenter_spec.rb'
-
'
spec/presenters/ci/trigger_presenter_spec.rb'
-
'
spec/presenters/packages/conan/package_presenter_spec.rb'
-
'
spec/requests/api/access_requests_spec.rb'
-
'
spec/requests/api/boards_spec.rb'
-
'
spec/requests/api/branches_spec.rb'
-
'
spec/requests/api/ci/runner_spec.rb'
-
'
spec/requests/api/commit_statuses_spec.rb'
-
'
spec/requests/api/conan_packages_spec.rb'
...
...
changelogs/unreleased/rails-save-bang-28.yml
0 → 100644
View file @
a0dd86b9
---
title
:
Fix Rails/SaveBang offenses for spec/requests/api/*
merge_request
:
41362
author
:
Rajendra Kadam
type
:
fixed
spec/requests/api/access_requests_spec.rb
View file @
a0dd86b9
...
...
@@ -90,7 +90,7 @@ RSpec.describe API::AccessRequests do
context
'when authenticated as a stranger'
do
context
"when access request is disabled for the
#{
source_type
}
"
do
before
do
source
.
update
(
request_access_enabled:
false
)
source
.
update
!
(
request_access_enabled:
false
)
end
it
'returns 403'
do
...
...
spec/requests/api/boards_spec.rb
View file @
a0dd86b9
...
...
@@ -41,7 +41,7 @@ RSpec.describe API::Boards do
it
'creates a new issue board list for group labels'
do
group
=
create
(
:group
)
group_label
=
create
(
:group_label
,
group:
group
)
board_parent
.
update
(
group:
group
)
board_parent
.
update
!
(
group:
group
)
post
api
(
url
,
user
),
params:
{
label_id:
group_label
.
id
}
...
...
@@ -54,7 +54,7 @@ RSpec.describe API::Boards do
group
=
create
(
:group
)
sub_group
=
create
(
:group
,
parent:
group
)
group_label
=
create
(
:group_label
,
group:
group
)
board_parent
.
update
(
group:
sub_group
)
board_parent
.
update
!
(
group:
sub_group
)
group
.
add_developer
(
user
)
sub_group
.
add_developer
(
user
)
...
...
spec/requests/api/branches_spec.rb
View file @
a0dd86b9
...
...
@@ -181,7 +181,7 @@ RSpec.describe API::Branches do
context
'when unauthenticated'
,
'and project is public'
do
before
do
project
.
update
(
visibility_level:
Gitlab
::
VisibilityLevel
::
PUBLIC
)
project
.
update
!
(
visibility_level:
Gitlab
::
VisibilityLevel
::
PUBLIC
)
end
it_behaves_like
'repository branches'
...
...
@@ -289,7 +289,7 @@ RSpec.describe API::Branches do
context
'when unauthenticated'
,
'and project is public'
do
before
do
project
.
update
(
visibility_level:
Gitlab
::
VisibilityLevel
::
PUBLIC
)
project
.
update
!
(
visibility_level:
Gitlab
::
VisibilityLevel
::
PUBLIC
)
end
it_behaves_like
'repository branch'
...
...
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