Don't disable recommended extensions for a base product

lib/rmt/cli/products.rb

@@ -52,11 +52,13 @@ def change_product(target, set_enabled)
       products = Product.where(conditions).to_a
     end
 
-    products.each do |product|
-      extensions = Product.recommended_extensions(product.id).to_a
-      next if extensions.empty?
-      puts "The following required extensions for #{product.product_string} have been enabled: #{extensions.pluck(:name).join(', ')}."
-      products.push(*extensions)
+    if set_enabled
+      products.each do |product|
+        extensions = Product.recommended_extensions(product.id).to_a
+        next if extensions.empty?
+        puts "The following required extensions for #{product.product_string} have been enabled: #{extensions.pluck(:name).join(', ')}."
+        products.push(*extensions)
+      end
     end
 
     repo_count = repository_service.change_mirroring_by_product!(set_enabled, products.uniq)

spec/lib/rmt/cli/products_spec.rb

@@ -129,7 +129,7 @@
         end
       end
 
-      context 'with recommended products' do
+      context 'with recommended extensions' do
         let(:product) { create :product, :with_not_mirrored_repositories, :with_recommended_extensions }
         let(:extensions) { Product.recommended_extensions(product).to_a }
         let(:products) { [product] + extensions }
@@ -198,6 +198,22 @@
           expect(repository.mirroring_enabled).to eq(false)
         end
       end
+
+      context 'with recommended extensions' do
+        let(:product) { create :product, :with_mirrored_repositories, :with_recommended_extensions }
+        let(:extensions) { Product.recommended_extensions(product).to_a }
+
+        it 'disabled the mandatory product repositories' do
+          product.repositories.each do |repository|
+            expect(repository.mirroring_enabled).to eq(false)
+          end
+          extensions.each do |extension|
+          extension.repositories.each do |repository|
+            expect(repository.mirroring_enabled).to eq(false)
+          end
+          end
+        end
+      end
     end
 
     context 'by product string' do