Skip to content

Commit 5e61627

Browse files
author
Vincent Febvre
committed
Describe further specs to highlight issue to fix
Signed-off-by: Vincent Febvre <[email protected]>
1 parent a490a07 commit 5e61627

File tree

2 files changed

+14
-5
lines changed

2 files changed

+14
-5
lines changed

spec/models/manageiq/providers/ibm_power_hmc/infra_manager/lpar_spec.rb

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,17 @@
2020
end
2121

2222
context "lpar" do
23+
let(:archived_vm) { FactoryBot.create(:ibm_power_hmc_lpar, :ext_management_system => nil, :ems_ref => "3F3D399B-DFF3-4977-8881-C194AA47CD3A", :host => host) }
24+
2325
it "supports clone" do
24-
expect(described_class.supports?(:clone)).to be false
26+
expect(vm.supports?(:clone)).to be false
2527
end
2628
it "supports publish" do
27-
expect(described_class.supports?(:publish)).to (be true), "unsupported reason: #{described_class.unsupported_reason(:publish)}"
29+
expect(vm.supports?(:publish)).to (be true), "unsupported reason: #{described_class.unsupported_reason(:publish)}"
30+
expect(archived_vm.supports?(:publish)).to be false
2831
end
2932
it "supports migrate" do
30-
expect(described_class.supports?(:migrate)).to be false
33+
expect(vm.supports?(:migrate)).to be false
3134
end
3235
it "supports power operations" do
3336
host.advanced_settings.create!(:name => "hmc_managed", :value => "true")
Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,16 @@
11
describe ManageIQ::Providers::IbmPowerHmc::InfraManager::Template do
2+
let(:ems) { FactoryBot.create(:ems_ibm_power_hmc_infra_with_authentication) }
3+
let(:template) { FactoryBot.create(:ibm_power_hmc_template, :ext_management_system => ems, :ems_ref => "template1_uuid", :name => "template1") }
4+
let(:archived_template) { FactoryBot.create(:ibm_power_hmc_template, :ext_management_system => nil, :ems_ref => "template2_uuid", :name => "template2") }
5+
26
context "template" do
37
it "supports clone" do
4-
expect(described_class.supports?(:clone)).to be true
8+
expect(template.supports?(:clone)).to (be true), "unsupported reason: #{described_class.unsupported_reason(:clone)}"
9+
expect(archived_template.supports?(:clone)).to be false
510
end
611
it "supports provisioning" do
7-
expect(described_class.supports?(:provisioning)).to be true
12+
expect(template.supports?(:provisioning)).to (be true), "unsupported reason: #{described_class.unsupported_reason(:provisioning)}"
13+
expect(archived_template.supports?(:provisioning)).to be false
814
end
915
end
1016
end

0 commit comments

Comments
 (0)