Use Environment#deployment_platform method instead

As this method does have `deployment_platform(environment: self.name)`,
which is what the form EE needs. Allows us to remove two overrides in
EE.
This commit is contained in:
Thong Kuah 2019-05-03 00:07:45 +12:00
parent a4777d6ea9
commit 0398213d20
2 changed files with 3 additions and 5 deletions

View File

@ -155,11 +155,11 @@ class Environment < ApplicationRecord
end
def has_terminals?
project.deployment_platform.present? && available? && last_deployment.present?
deployment_platform.present? && available? && last_deployment.present?
end
def terminals
project.deployment_platform.terminals(self) if has_terminals?
deployment_platform.terminals(self) if has_terminals?
end
def has_metrics?

View File

@ -592,9 +592,7 @@ describe Environment do
shared_examples 'same behavior between KubernetesService and Platform::Kubernetes' do
it 'returns the terminals from the deployment service' do
deployment_platform_target = Gitlab.ee? ? environment : project
expect(deployment_platform_target.deployment_platform)
expect(environment.deployment_platform)
.to receive(:terminals).with(environment)
.and_return(:fake_terminals)