Commit 16e645c6 authored by Grzegorz Bizon's avatar Grzegorz Bizon

Remove container_registry method from project class

parent 4fdcaca6
...@@ -405,20 +405,6 @@ class Project < ActiveRecord::Base ...@@ -405,20 +405,6 @@ class Project < ActiveRecord::Base
@repository ||= Repository.new(path_with_namespace, self) @repository ||= Repository.new(path_with_namespace, self)
end end
def container_registry
return unless Gitlab.config.registry.enabled
@container_registry ||= begin
token = Auth::ContainerRegistryAuthenticationService.full_access_token(project)
url = Gitlab.config.registry.api_url
host_port = Gitlab.config.registry.host_port
# TODO, move configuration vars into ContainerRegistry::Registry, clean
# this method up afterwards
ContainerRegistry::Registry.new(url, token: token, path: host_port)
end
end
def container_registry_url def container_registry_url
if Gitlab.config.registry.enabled if Gitlab.config.registry.enabled
"#{Gitlab.config.registry.host_port}/#{path_with_namespace.downcase}" "#{Gitlab.config.registry.host_port}/#{path_with_namespace.downcase}"
......
...@@ -1389,25 +1389,6 @@ describe Project, models: true do ...@@ -1389,25 +1389,6 @@ describe Project, models: true do
end end
end end
describe '#container_registry_path_with_namespace' do
let(:project) { create(:empty_project, path: 'PROJECT') }
subject { project.container_registry_path_with_namespace }
it { is_expected.not_to eq(project.path_with_namespace) }
it { is_expected.to eq(project.path_with_namespace.downcase) }
end
describe '#container_registry' do
let(:project) { create(:empty_project) }
before { stub_container_registry_config(enabled: true) }
subject { project.container_registry }
it { is_expected.not_to be_nil }
end
describe '#container_registry_url' do describe '#container_registry_url' do
let(:project) { create(:empty_project) } let(:project) { create(:empty_project) }
...@@ -1417,10 +1398,8 @@ describe Project, models: true do ...@@ -1417,10 +1398,8 @@ describe Project, models: true do
context 'for enabled registry' do context 'for enabled registry' do
let(:registry_settings) do let(:registry_settings) do
{ { enabled: true,
enabled: true, host_port: 'example.com' }
host_port: 'example.com',
}
end end
it { is_expected.not_to be_nil } it { is_expected.not_to be_nil }
...@@ -1428,9 +1407,7 @@ describe Project, models: true do ...@@ -1428,9 +1407,7 @@ describe Project, models: true do
context 'for disabled registry' do context 'for disabled registry' do
let(:registry_settings) do let(:registry_settings) do
{ { enabled: false }
enabled: false
}
end end
it { is_expected.to be_nil } it { is_expected.to be_nil }
......
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