Commit f5e37ecb authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'leaky-constant-fix-8' into 'master'

Fix leaky constant in upgrade progess service check

See merge request gitlab-org/gitlab!31969
parents e5ee54cb b97eb0b2
...@@ -396,8 +396,6 @@ RSpec/LeakyConstantDeclaration: ...@@ -396,8 +396,6 @@ RSpec/LeakyConstantDeclaration:
- 'spec/services/ci/retry_build_service_spec.rb' - 'spec/services/ci/retry_build_service_spec.rb'
- 'spec/services/clusters/applications/check_installation_progress_service_spec.rb' - 'spec/services/clusters/applications/check_installation_progress_service_spec.rb'
- 'spec/services/clusters/applications/check_uninstall_progress_service_spec.rb' - 'spec/services/clusters/applications/check_uninstall_progress_service_spec.rb'
- 'spec/services/clusters/applications/check_upgrade_progress_service_spec.rb'
- 'spec/services/clusters/applications/ingress_modsecurity_usage_service_spec.rb'
- 'spec/services/issues/resolve_discussions_spec.rb' - 'spec/services/issues/resolve_discussions_spec.rb'
- 'spec/support/shared_contexts/spam_constants.rb' - 'spec/support/shared_contexts/spam_constants.rb'
- 'spec/support/shared_examples/quick_actions/issuable/issuable_quick_actions_shared_examples.rb' - 'spec/support/shared_examples/quick_actions/issuable/issuable_quick_actions_shared_examples.rb'
......
---
title: Fix leaky constant issue in upgrade progress service check
merge_request: 31969
author: Rajendra Kadam
type: fixed
...@@ -3,7 +3,7 @@ ...@@ -3,7 +3,7 @@
require 'spec_helper' require 'spec_helper'
describe Clusters::Applications::CheckUpgradeProgressService do describe Clusters::Applications::CheckUpgradeProgressService do
RESCHEDULE_PHASES = ::Gitlab::Kubernetes::Pod::PHASES - reschedule_phashes = ::Gitlab::Kubernetes::Pod::PHASES -
[::Gitlab::Kubernetes::Pod::SUCCEEDED, ::Gitlab::Kubernetes::Pod::FAILED, ::Gitlab].freeze [::Gitlab::Kubernetes::Pod::SUCCEEDED, ::Gitlab::Kubernetes::Pod::FAILED, ::Gitlab].freeze
let(:application) { create(:clusters_applications_prometheus, :updating) } let(:application) { create(:clusters_applications_prometheus, :updating) }
...@@ -89,6 +89,6 @@ describe Clusters::Applications::CheckUpgradeProgressService do ...@@ -89,6 +89,6 @@ describe Clusters::Applications::CheckUpgradeProgressService do
end end
end end
RESCHEDULE_PHASES.each { |phase| it_behaves_like 'a not yet terminated upgrade', phase } reschedule_phashes.each { |phase| it_behaves_like 'a not yet terminated upgrade', phase }
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