Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKamil Trzcinski <ayufan@ayufan.eu>2016-05-15 18:46:54 +0300
committerKamil Trzcinski <ayufan@ayufan.eu>2016-05-15 18:46:54 +0300
commit938d5b6fe4bb3d7d6d85188fc5fd1aac77803577 (patch)
treea4e40b2686b9491147ba346161bec948fe2bb616 /spec/services/auth
parent3d18b3a0e7e69924d6a5cf39b180b1b133f8c9b0 (diff)
Fix http status codes for container registry authentication service
Diffstat (limited to 'spec/services/auth')
-rw-r--r--spec/services/auth/container_registry_authentication_service_spec.rb17
1 files changed, 6 insertions, 11 deletions
diff --git a/spec/services/auth/container_registry_authentication_service_spec.rb b/spec/services/auth/container_registry_authentication_service_spec.rb
index 409d08e49f1..143d992b6e4 100644
--- a/spec/services/auth/container_registry_authentication_service_spec.rb
+++ b/spec/services/auth/container_registry_authentication_service_spec.rb
@@ -57,11 +57,6 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
end
end
- shared_examples 'a unauthorized' do
- it { is_expected.to include(http_status: 401) }
- it { is_expected.to_not include(:token) }
- end
-
shared_examples 'a forbidden' do
it { is_expected.to include(http_status: 403) }
it { is_expected.to_not include(:token) }
@@ -116,7 +111,7 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
{ scope: "repository:#{project.path_with_namespace}:pull,push" }
end
- it_behaves_like 'a unauthorized'
+ it_behaves_like 'a forbidden'
end
end
@@ -154,7 +149,7 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
context 'disallow for private' do
let(:project) { create(:empty_project, :private) }
- it_behaves_like 'a unauthorized'
+ it_behaves_like 'a forbidden'
end
end
@@ -165,7 +160,7 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
context 'disallow for all' do
let(:project) { create(:empty_project, :public) }
- it_behaves_like 'a unauthorized'
+ it_behaves_like 'a forbidden'
end
end
end
@@ -199,7 +194,7 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
{ scope: 'invalid:aa:bb' }
end
- it_behaves_like 'a unauthorized'
+ it_behaves_like 'a forbidden'
end
context 'for private project' do
@@ -209,7 +204,7 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
{ scope: "repository:#{project.path_with_namespace}:pull" }
end
- it_behaves_like 'a unauthorized'
+ it_behaves_like 'a forbidden'
end
context 'for public project' do
@@ -228,7 +223,7 @@ describe Auth::ContainerRegistryAuthenticationService, services: true do
{ scope: "repository:#{project.path_with_namespace}:push" }
end
- it_behaves_like 'a unauthorized'
+ it_behaves_like 'a forbidden'
end
end
end