Fix markup (and therefore styling) of RTL history mode banner

Right-to-left languages were displaying the government name in
an unusual way, treating the government name as translated text
even though it is still English. This commit fixes the text
direction, as well as setting the correct language for the text
snippet.

Before:
![to 2024 Sunak Conservative government 2022](https://github.com/user-attachments/assets/999781ba-8f6e-4c03-8f6d-00d9b8921c4c)

After:
![2022 to 2024 Sunak Conservative government](https://github.com/user-attachments/assets/d94c7979-03f3-46ca-bc97-4a86951a8b31)

app/presenters/content_item/political.rb

@@ -5,9 +5,10 @@ def historically_political?
     end
 
     def publishing_government
-      content_item.dig(
+      str = content_item.dig(
         "links", "government", 0, "title"
       )
+      "<span lang='en' dir='ltr'>#{str}</span>"
     end
 
   private

app/views/shared/_history_notice.html.erb

@@ -1,5 +1,8 @@
 <% if content_item.historically_political? %>
   <%= render "govuk_publishing_components/components/notice", {
-    title: t("shared.historically_political", government: content_item.publishing_government),
+    title: sanitize(
+      t("shared.historically_political", government: content_item.publishing_government),
+      attributes: %w(lang dir),
+    ),
   } %>
 <% end %>

test/integration/news_article_test.rb

@@ -42,6 +42,14 @@ class NewsArticleTest < ActionDispatch::IntegrationTest
     end
   end
 
+  test "marks up government name correctly" do
+    setup_and_visit_content_item("news_article_history_mode_translated_arabic")
+
+    within ".govuk-notification-banner__content" do
+      assert page.has_css?("span[lang='en'][dir='ltr']", text: "2022 to 2024 Sunak Conservative government")
+    end
+  end
+
   test "does not render with the single page notification button" do
     setup_and_visit_content_item("news_article")
     assert_not page.has_css?(".gem-c-single-page-notification-button")

test/presenters/detailed_guide_presenter_test.rb

@@ -32,7 +32,7 @@ def schema_name
   end
 
   test "presents the title of the publishing government" do
-    assert_equal schema_item["details"]["government"]["title"], presented_item.publishing_government
+    assert_equal "<span lang='en' dir='ltr'>#{schema_item['details']['government']['title']}", presented_item.publishing_government
   end
 
   test "content can be historically political" do

test/presenters/publication_presenter_test.rb

@@ -17,7 +17,7 @@ def schema_name
   end
 
   test "presents the title of the publishing government" do
-    assert_equal schema_item["links"]["government"][0]["title"], presented_item.publishing_government
+    assert_equal "<span lang='en' dir='ltr'>#{schema_item['links']['government'][0]['title']}</span>", presented_item.publishing_government
   end
 
   test "content can be historically political" do

test/presenters/statistical_data_set_presenter_test.rb

@@ -76,7 +76,7 @@ def presented
 
     test "presents the political fields" do
       assert_equal expected["details"]["political"], presented.historically_political?
-      assert_equal expected["details"]["government"]["title"], presented.publishing_government
+      assert_equal "<span lang='en' dir='ltr'>#{expected['details']['government']['title']}</span>", presented.publishing_government
     end
   end
 end