Don't run translations if not set (#396)

lib/tasks/repair_data.rake

@@ -39,18 +39,22 @@ namespace :decidim do
     desc "Add all missing translation for translatable resources"
     task translations: :environment do
       logger = Logger.new($stdout)
-      logger.info("Checking all translatable resources...")
+      if Decidim.enable_machine_translations
+        logger.info("Checking all translatable resources...")
 
-      updated_resources_ids = Decidim::RepairTranslationsService.run(logger: logger)
+        updated_resources_ids = Decidim::RepairTranslationsService.run(logger: logger)
 
-      if updated_resources_ids.blank?
-        logger.info("No resources updated")
+        if updated_resources_ids.blank?
+          logger.info("No resources updated")
+        else
+          logger.info("#{updated_resources_ids.count} resources enqueue for translation")
+          logger.info("Enqueued resources : #{updated_resources_ids.join(", ")}")
+        end
+
+        logger.info("Operation terminated")
       else
-        logger.info("#{updated_resources_ids.count} resources enqueue for translation")
-        logger.info("Enqueued resources : #{updated_resources_ids.join(", ")}")
+        logger.warn("Machine translation is not enabled")
       end
-
-      logger.info("Operation terminated")
     end
 
     task url_in_content: :environment do

spec/lib/tasks/repair_data_translations_spec.rb

@@ -5,10 +5,12 @@
 describe "rake decidim:repair:translations", type: :task do
   let(:task) { Rake::Task[:"decidim:repair:translations"] }
   let!(:comment) { create(:comment) }
+  let(:enable_machine_translations) { true }
 
   before do
     clear_enqueued_jobs
     clear_performed_jobs
+    allow(Decidim).to receive(:enable_machine_translations).and_return(enable_machine_translations)
   end
 
   after do
@@ -25,4 +27,14 @@
 
     task.execute
   end
+
+  context "when translation is not activated" do
+    let(:enable_machine_translations) { false }
+
+    it "doesn't calls the service" do
+      expect(Decidim::RepairTranslationsService).not_to receive(:run)
+
+      task.execute
+    end
+  end
 end