]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Remove unused, outdated unit test cases
authorTakashi Kajinami <tkajinam@redhat.com>
Mon, 8 Nov 2021 14:24:23 +0000 (23:24 +0900)
committerTakashi Kajinami <tkajinam@redhat.com>
Mon, 8 Nov 2021 14:27:09 +0000 (23:27 +0900)
These test cases were added when the metering_secret parameter was
deprecated in favor of the new telemetry_secret parameter[1], but were
not enabled properly.

The deprecated metering_secret parameter was later removed[2] but
these ineffective test cases were not removed at that time.

[1] 4600d4135ac1bef01f7a7dd4b12582a0c8db6b50
[2] f28830d5aae8021215026bdf2f63a8713b7deeb1

Change-Id: I80290f4388e9bb3eb5542a9d1e70ccea81698905

spec/classes/ceilometer_init_spec.rb

index 1b0834354eceaccf1a7557e12cff5fee61a59a59..a0eccd3cfc8d8aa5d74ab1543db1813834a2b202 100644 (file)
@@ -250,25 +250,6 @@ describe 'ceilometer' do
     end
   end
 
-
-  # Cleanup in Ocata
-  shared_examples_for 'using old metering_secret param' do
-    context "with old metering_secret param it uses telemetry_secret instead" do
-      before { params.merge!(
-          :metering_secret => 'broncos',
-          :telemetry_secret => 'metering-s3cr3t',
-      ) }
-      it { is_expected.to contain_ceilometer_config('publisher/telemetry_secret').with_value('metering-s3cr3t') }
-    end
-    context "with old metering_secret param set and telemetry_secret unset" do
-      before { params.merge!(
-          :metering_secret => 'broncos',
-          :telemetry_secret => nil,
-      ) }
-      it { is_expected.to contain_ceilometer_config('publisher/telemetry_secret').with_value('broncos') }
-    end
-  end
-
   shared_examples_for 'rabbit with SSL support' do
     context "with default parameters" do
     it { is_expected.to contain_oslo__messaging__rabbit('ceilometer_config').with(