FIX: Update localisation instead of category

Author: nattsw

app/services/discourse_translator/category_translator.rb

@@ -16,7 +16,16 @@ def self.translate(category, target_locale = I18n.locale)
       translated_name = translator.translate_text!(category.name, target_locale_sym)
       translated_description = translator.translate_text!(category.description, target_locale_sym)
 
-      category.update!(name: translated_name, description: translated_description)
+      localization =
+        CategoryLocalization.find_or_initialize_by(
+          category_id: category.id,
+          locale: target_locale_sym.to_s,
+        )
+
+      localization.name = translated_name
+      localization.description = translated_description
+      localization.save!
+      localization
     end
   end
 end

spec/services/category_translator_spec.rb

@@ -21,10 +21,35 @@
         .with(category.description, target_locale)
         .returns("C'est une catégorie de test")
 
+      res = DiscourseTranslator::CategoryTranslator.translate(category, target_locale)
+
+      expect(res.name).to eq("Catégorie de Test")
+      expect(res.description).to eq("C'est une catégorie de test")
+    end
+
+    it "translates the category name and description" do
+      localized =
+        Fabricate(
+          :category_localization,
+          category: category,
+          locale: target_locale,
+          name: "X",
+          description: "Y",
+        )
+      translator
+        .expects(:translate_text!)
+        .with(category.name, target_locale)
+        .returns("Catégorie de Test")
+      translator
+        .expects(:translate_text!)
+        .with(category.description, target_locale)
+        .returns("C'est une catégorie de test")
+
       DiscourseTranslator::CategoryTranslator.translate(category, target_locale)
 
-      expect(category.name).to eq("Catégorie de Test")
-      expect(category.description).to eq("C'est une catégorie de test")
+      localized.reload
+      expect(localized.name).to eq("Catégorie de Test")
+      expect(localized.description).to eq("C'est une catégorie de test")
     end
 
     it "handles locale format standardization" do