Merge pull request #17866 from mansam/cleanup-mapped-tenants-after-em…

…s-destroy

Clean up mapped tenants after a CloudManager is destroyed

(cherry picked from commit 89a1ebf)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1547740

app/models/manageiq/providers/cloud_manager.rb

@@ -42,6 +42,8 @@ class << model_name
     include HasNetworkManagerMixin
     include HasManyOrchestrationStackMixin
 
+    after_destroy :destroy_mapped_tenants
+
     # Development helper method for Rails console for opening a browser to the EMS.
     #
     # This method is NOT meant to be called from production code.
@@ -142,5 +144,13 @@ def create_cloud_tenant(options)
     def self.display_name(number = 1)
       n_('Cloud Manager', 'Cloud Managers', number)
     end
+
+    def destroy_mapped_tenants
+      if source_tenant
+        source_tenant.all_subtenants.destroy_all
+        source_tenant.all_subprojects.destroy_all
+        source_tenant.destroy
+      end
+    end
   end
 end

spec/models/manageiq/providers/cloud_manager_spec.rb

@@ -192,6 +192,17 @@ def expect_created_tenant_tree
           expect_created_tenant_tree
         end
 
+        it "cleans up created tenant tree when the ems is destroyed" do
+          ems_cloud.sync_cloud_tenants_with_tenants
+          expect_created_tenant_tree
+          # 4 cloud tenants, plus the provider tenant and root tenant
+          expect(Tenant.count).to eq(6)
+          ems_cloud.destroy
+          # only the root tenant should remain after destroying the ems
+          expect(Tenant.count).to eq(1)
+          expect(Tenant.first.name).to eq("My Company")
+        end
+
         let(:vm_5) { FactoryGirl.create(:vm_openstack) }
 
         let(:ct_5) do