Commit 760a154a authored by Kamil Trzcinski's avatar Kamil Trzcinski

Fix tests for initial status

parent 55d098c9
...@@ -18,6 +18,8 @@ module Clusters ...@@ -18,6 +18,8 @@ module Clusters
end end
def set_initial_status def set_initial_status
return unless not_installable?
self.status = 'installable' if cluster&.platform_kubernetes_active? self.status = 'installable' if cluster&.platform_kubernetes_active?
end end
......
...@@ -2,6 +2,10 @@ FactoryGirl.define do ...@@ -2,6 +2,10 @@ FactoryGirl.define do
factory :cluster_applications_helm, class: Clusters::Applications::Helm do factory :cluster_applications_helm, class: Clusters::Applications::Helm do
cluster factory: %i(cluster provided_by_gcp) cluster factory: %i(cluster provided_by_gcp)
trait :not_installable do
status -2
end
trait :installable do trait :installable do
status 0 status 0
end end
......
...@@ -21,8 +21,20 @@ describe Clusters::Applications::Helm do ...@@ -21,8 +21,20 @@ describe Clusters::Applications::Helm do
end end
describe '#status' do describe '#status' do
it 'defaults to :installable' do let(:cluster) { create(:cluster) }
expect(subject.status_name).to be(:installable)
subject { described_class.new(cluster: cluster) }
it 'defaults to :not_installable' do
expect(subject.status_name).to be(:not_installable)
end
context 'when platform kubernetes is defined' do
let(:cluster) { create(:cluster, :provided_by_gcp) }
it 'defaults to :installable' do
expect(subject.status_name).to be(:installable)
end
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