Localise model validation messages and refusal reasons

2 files changed, 18 insertions, 1 deletions

diff --git a/test/models/node_test.rb b/test/models/node_test.rb
index c5b0d6a2..c1316ea6 100644
--- a/test/models/node_test.rb
+++ b/test/models/node_test.rb
@@ -919,4 +919,21 @@ class NodeTest < ActiveSupport::TestCase
     node.reload
     assert_raises(ActiveRecord::RecordInvalid) { node.publish_draft! }
   end
+
+  test "refusals localise their message rather than hardcoding it" do
+    trash = Node.trash
+
+    I18n.with_locale(:de) do
+      error = assert_raises(ActiveRecord::RecordInvalid) { trash.trash! }
+      assert_includes error.message,
+                      I18n.t("activerecord.errors.models.node.attributes.base.trash_the_trash")
+    end
+
+    I18n.with_locale(:en) do
+      trash.errors.clear
+      error = assert_raises(ActiveRecord::RecordInvalid) { trash.trash! }
+      assert_includes error.message,
+                      I18n.t("activerecord.errors.models.node.attributes.base.trash_the_trash")
+    end
+  end
 end
diff --git a/test/models/related_asset_test.rb b/test/models/related_asset_test.rb
index bb86ddbb..ea7c3d35 100644
--- a/test/models/related_asset_test.rb
+++ b/test/models/related_asset_test.rb
@@ -29,7 +29,7 @@ class RelatedAssetTest < ActiveSupport::TestCase
 
     related.headline = true
     assert_not related.valid?
-    assert_includes related.errors[:headline], "can only be set on image or PDF assets"
+    assert_includes related.errors[:headline], I18n.t("activerecord.errors.models.related_asset.attributes.headline.images_and_pdfs_only")
   end
 
   test "the headline validation does not raise when asset is missing" do