Commit 1c404c91 authored by Shinya Maeda's avatar Shinya Maeda

Add a new test for emptified params

parent 2d3c7d29
...@@ -42,8 +42,8 @@ describe MigrateKubernetesServiceToNewClustersArchitectures, :migration do ...@@ -42,8 +42,8 @@ describe MigrateKubernetesServiceToNewClustersArchitectures, :migration do
context 'when KubernetesService is not active' do context 'when KubernetesService is not active' do
let(:active) { false } let(:active) { false }
# Platforms::Kubernetes validates `token` reagdless of the activeness # Platforms::Kubernetes validates `token` reagdless of the activeness,
# KubernetesService validates `token` if only it's activated # whereas KubernetesService validates `token` if only it's activated
# However, in this migration file, there are no validations because of the migration specific model class # However, in this migration file, there are no validations because of the migration specific model class
# therefore, Validation Error will not happen in this case and just migrate data # therefore, Validation Error will not happen in this case and just migrate data
let(:token) { '' } let(:token) { '' }
...@@ -169,6 +169,23 @@ describe MigrateKubernetesServiceToNewClustersArchitectures, :migration do ...@@ -169,6 +169,23 @@ describe MigrateKubernetesServiceToNewClustersArchitectures, :migration do
end end
end end
context 'when KubernetesService has nullified parameters' do
let(:project) { create(:project) }
before do
ActiveRecord::Base.connection.execute <<~SQL
INSERT INTO services (project_id, active, category, type, properties)
VALUES (#{project.id}, false, 'deployment', 'KubernetesService', '{}');
SQL
end
it 'does not migrate the KubernetesService and disables the kubernetes_service' do
expect { migrate! }.not_to change { Clusters::Cluster.count }
expect(project.kubernetes_service).not_to be_active
end
end
context 'when KubernetesService does not exist' do context 'when KubernetesService does not exist' do
let!(:project) { create(:project) } let!(:project) { create(:project) }
......
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