Commit 5e1a685d authored by Alessio Caiazza's avatar Alessio Caiazza

Merge branch 'pl-rubocop-have-gitlab-http-status-spec-features' into 'master'

Enable cop HaveGitlabHttpStatus in feature specs

See merge request gitlab-org/gitlab!23554
parents be8cb784 b78950e5
...@@ -348,10 +348,11 @@ RSpec/HaveGitlabHttpStatus: ...@@ -348,10 +348,11 @@ RSpec/HaveGitlabHttpStatus:
Include: Include:
- 'spec/support/shared_examples/**/*' - 'spec/support/shared_examples/**/*'
- 'ee/spec/support/shared_examples/**/*' - 'ee/spec/support/shared_examples/**/*'
- 'spec/features/**/*'
- 'ee/spec/features/**/*'
Style/MultilineWhenThen: Style/MultilineWhenThen:
Enabled: false Enabled: false
Style/FloatDivision: Style/FloatDivision:
Enabled: false Enabled: false
\ No newline at end of file
...@@ -91,7 +91,7 @@ describe 'Group Dependency Proxy' do ...@@ -91,7 +91,7 @@ describe 'Group Dependency Proxy' do
visit path visit path
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
...@@ -101,7 +101,7 @@ describe 'Group Dependency Proxy' do ...@@ -101,7 +101,7 @@ describe 'Group Dependency Proxy' do
visit path visit path
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
end end
......
...@@ -28,7 +28,7 @@ describe 'Group Packages' do ...@@ -28,7 +28,7 @@ describe 'Group Packages' do
it 'gives 404' do it 'gives 404' do
visit_group_packages visit_group_packages
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
...@@ -40,7 +40,7 @@ describe 'Group Packages' do ...@@ -40,7 +40,7 @@ describe 'Group Packages' do
it 'gives 404' do it 'gives 404' do
visit_group_packages visit_group_packages
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
......
...@@ -54,7 +54,7 @@ describe 'Groups > Usage Quotas' do ...@@ -54,7 +54,7 @@ describe 'Groups > Usage Quotas' do
it 'renders a 404' do it 'renders a 404' do
visit_pipeline_quota_page visit_pipeline_quota_page
expect(page).to have_http_status(:not_found) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
end end
...@@ -151,7 +151,7 @@ describe 'Groups > Usage Quotas' do ...@@ -151,7 +151,7 @@ describe 'Groups > Usage Quotas' do
it 'does not show subproject' do it 'does not show subproject' do
visit_pipeline_quota_page visit_pipeline_quota_page
expect(page).to have_http_status(:not_found) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
......
...@@ -21,7 +21,7 @@ describe 'User activates GitHub Service' do ...@@ -21,7 +21,7 @@ describe 'User activates GitHub Service' do
it 'renders 404 when trying to access service settings directly' do it 'renders 404 when trying to access service settings directly' do
visit edit_project_service_path(project, :github) visit edit_project_service_path(project, :github)
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
......
...@@ -28,7 +28,7 @@ describe 'Protected Environments' do ...@@ -28,7 +28,7 @@ describe 'Protected Environments' do
end end
it 'does not have access to Protected Environments settings' do it 'does not have access to Protected Environments settings' do
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
...@@ -40,7 +40,7 @@ describe 'Protected Environments' do ...@@ -40,7 +40,7 @@ describe 'Protected Environments' do
end end
it 'has access to Protected Environments settings' do it 'has access to Protected Environments settings' do
expect(page).to have_gitlab_http_status(200) expect(page).to have_gitlab_http_status(:ok)
end end
it 'allows seeing a list of protected environments' do it 'allows seeing a list of protected environments' do
...@@ -103,7 +103,7 @@ describe 'Protected Environments' do ...@@ -103,7 +103,7 @@ describe 'Protected Environments' do
it 'shows setting page correctly' do it 'shows setting page correctly' do
visit project_settings_ci_cd_path(project) visit project_settings_ci_cd_path(project)
expect(page).to have_gitlab_http_status(200) expect(page).to have_gitlab_http_status(:ok)
end end
end end
end end
......
...@@ -63,7 +63,7 @@ describe 'Profile > Password' do ...@@ -63,7 +63,7 @@ describe 'Profile > Password' do
visit edit_profile_password_path visit edit_profile_password_path
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
...@@ -73,7 +73,7 @@ describe 'Profile > Password' do ...@@ -73,7 +73,7 @@ describe 'Profile > Password' do
it 'renders 404' do it 'renders 404' do
visit edit_profile_password_path visit edit_profile_password_path
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
end end
......
...@@ -54,7 +54,7 @@ describe 'test coverage badge' do ...@@ -54,7 +54,7 @@ describe 'test coverage badge' do
it 'user requests test coverage badge image' do it 'user requests test coverage badge image' do
show_test_coverage_badge show_test_coverage_badge
expect(page).to have_gitlab_http_status(404) expect(page).to have_gitlab_http_status(:not_found)
end end
end end
......
...@@ -7,7 +7,7 @@ describe 'User views tags', :feature do ...@@ -7,7 +7,7 @@ describe 'User views tags', :feature do
it do it do
visit project_tags_path(project, format: :atom) visit project_tags_path(project, format: :atom)
expect(page).to have_gitlab_http_status(200) expect(page).to have_gitlab_http_status(:ok)
end end
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