Update views to use the new popular_list method.

app/helpers/browse_helper.rb

@@ -1,21 +1,10 @@
 module BrowseHelper
-  def display_popular_links_for_slug?(slug)
-    popular_links_data(slug).present?
-  end
-
-  def popular_links_data(slug)
-    if I18n.exists?("browse.popular_links.#{slug}", :en)
-      I18n.t("browse.popular_links.#{slug}")
-    end
-  end
-
-  def popular_links_for_slug(slug)
-    links = popular_links_data(slug)
-    count = links.length
-    links.map.with_index(1) do |link, index|
+  def with_tracking_data(popular_content)
+    count = popular_content.length
+    popular_content.map.with_index(1) do |link, index|
       {
         text: link[:title],
-        href: link[:url],
+        href: link[:link],
         data_attributes: {
           module: "ga4-link-tracker",
           ga4_track_links_only: "",

app/views/browse/show.html.erb

@@ -34,7 +34,7 @@
   } %>
 <% end %>
 
-<% if display_popular_links_for_slug?(page.slug) %>
+<% if page.popular_list %>
   <div class="govuk-width-container">
     <div class="browse__action-links">
       <div class="govuk-grid-row">
@@ -45,7 +45,7 @@
             font_size: "m"
           } %>
           <ul class="govuk-list govuk-!-margin-bottom-7">
-            <% popular_links_for_slug(page.slug).each do |link| %>
+            <% with_tracking_data(page.popular_list).each do |link| %>
               <li>
                 <%= render partial: "shared/browse_action_link", locals: {link:} %>
               </li>
@@ -68,7 +68,7 @@
 <% total_links = page.second_level_browse_pages.count.to_s %>
 <%= render "shared/browse_cards_container" do %>
   <%= render "govuk_publishing_components/components/cards", {
-    heading: display_popular_links_for_slug?(page.slug) ? t("browse.topics") : nil,
+    heading: page.popular_list ? t("browse.topics") : nil,
     items: page.second_level_browse_pages.map.with_index do |second_level_browse_page, index|
       {
         link: {
@@ -86,6 +86,6 @@
         description: second_level_browse_page.description,
       }
     end,
-    sub_heading_level: display_popular_links_for_slug?(page.slug) ? 3 : 2,
+    sub_heading_level: page.popular_list ? 3 : 2,
   } %>
 <% end %>

features/step_definitions/viewing_browse_steps.rb

@@ -19,6 +19,17 @@
     ],
     page_size: SearchApiSearch::PAGE_SIZE_TO_GET_EVERYTHING,
   )
+
+  level_one_browse_content_item = {
+    content_id: "content-id-for-justice",
+    title: "Crime and justice",
+    base_path: "/browse/justice",
+    links: {
+      top_level_browse_pages:,
+      second_level_browse_pages:,
+    },
+  }
+  search_api_has_popular_documents_for_level_one_browse(level_one_browse_content_item)
 end
 
 Given(/^that there are curated second level browse pages$/) do
@@ -41,6 +52,17 @@
     order_type: "curated",
   )
   add_second_level_browse_pages(second_level_browse_pages)
+
+  level_one_browse_content_item = {
+    content_id: "content-id-for-justice",
+    title: "Crime and justice",
+    base_path: "/browse/justice",
+    links: {
+      top_level_browse_pages:,
+      second_level_browse_pages:,
+    },
+  }
+  search_api_has_popular_documents_for_level_one_browse(level_one_browse_content_item)
 end
 
 Then(/^I see the links tagged to the browse page/) do

spec/controllers/browse_controller_spec.rb

@@ -1,5 +1,7 @@
 RSpec.describe BrowseController do
   include GovukAbTesting::RspecHelpers
+  include SearchApiHelpers
+
   render_views
   describe "GET index" do
     before do
@@ -21,15 +23,19 @@
   describe "GET top_level_browse_page" do
     describe "for a valid browse page" do
       before do
-        stub_content_store_has_item(
-          "/browse/benefits",
+        content_item = {
           base_path: "/browse/benefits",
           title: "foo",
           links: {
             top_level_browse_pages:,
             second_level_browse_pages:,
           },
+        }
+        stub_content_store_has_item(
+          "/browse/benefits",
+          content_item,
         )
+        search_api_has_popular_documents_for_level_one_browse(content_item)
       end
 
       it "sets correct expiry headers" do

spec/features/mainstream_browsing_spec.rb

@@ -22,49 +22,52 @@
         ],
         page_size: SearchApiSearch::PAGE_SIZE_TO_GET_EVERYTHING,
       )
-
+      search_api_has_popular_documents_for_level_one_browse(content_item)
       visit content_item["base_path"]
 
       expect(page.status_code).to eq(200)
       expect(page).to have_selector(".gem-c-breadcrumbs")
     end
   end
 
-  it "renders popular tasks on level 1 browse pages" do
-    level_one_browse_slugs = %w[  abroad
-                                  benefits
-                                  births-deaths-marriages
-                                  business
-                                  childcare-parenting
-                                  citizenship
-                                  disabilities
-                                  driving
-                                  education
-                                  employing-people
-                                  environment-countryside
-                                  housing-local-services
-                                  justice
-                                  tax
-                                  visas-immigration
-                                  working]
-
-    level_one_browse_slugs.each do |slug|
-      content_item = GovukSchemas::Example.find("mainstream_browse_page", example_name: "top_level_page").tap do |item|
-        item["base_path"] = "/browse/#{slug}"
-      end
+  describe "popular tasks" do
+    it "render on level one browse pages" do
+      content_item = GovukSchemas::Example.find("mainstream_browse_page", example_name: "top_level_page")
       stub_content_store_has_item(content_item["base_path"], content_item)
+      search_api_has_popular_documents_for_level_one_browse(content_item)
+
       visit content_item["base_path"]
       expect(page).to have_css(".browse__action-links")
       expect(page).to have_content("Popular tasks")
     end
   end
 
+  it "do not render on the root browse page" do
+    content_item = GovukSchemas::Example.find("mainstream_browse_page", example_name: "root_page")
+    stub_content_store_has_item(content_item["base_path"], content_item)
+
+    visit content_item["base_path"]
+    expect(page).not_to have_css(".browse__action-links")
+    expect(page).not_to have_content("Popular tasks")
+  end
+
+  it "do not render on level two browse pages" do
+    content_item = GovukSchemas::Example.find("mainstream_browse_page", example_name: "level_2_page")
+    stub_content_store_has_item(content_item["base_path"], content_item)
+    search_api_has_documents_for_browse_page(content_item["content_id"], %w[a-slug], page_size: 1000)
+
+    visit content_item["base_path"]
+    expect(page).not_to have_css(".browse__action-links")
+    expect(page).not_to have_content("Popular tasks")
+  end
+
   it "renders the GOV.UK Chat promo" do
     content_item = GovukSchemas::Example.find("mainstream_browse_page", example_name: "top_level_page").tap do |item|
       item["base_path"] = GovukChatPromoHelper::GOVUK_CHAT_PROMO_BASE_PATHS.first
     end
 
     stub_content_store_has_item(content_item["base_path"], content_item)
+    search_api_has_popular_documents_for_level_one_browse(content_item)
 
     ClimateControl.modify GOVUK_CHAT_PROMO_ENABLED: "true" do
       visit content_item["base_path"]

spec/helpers/browse_helper_spec.rb

@@ -1,20 +1,25 @@
 RSpec.describe BrowseHelper do
-  describe "#display_popular_links_for_slug?" do
-    it "returns true if data exists" do
-      expect(helper.display_popular_links_for_slug?("business")).to be(true)
-    end
-
-    it "returns false if no data exists" do
-      expect(helper.display_popular_links_for_slug?("random12345")).to be(false)
-    end
-  end
+  describe "#with_tracking_data" do
+    it "returns formatted popular links" do
+      popular_content_from_search = [
+        {
+          'link': "/foo/policy_paper",
+          'title': "Policy on World Locations",
+        },
+        {
+          'link': "/foo/news_story",
+          'title': "PM attends summit on world location news pages",
+        },
+        {
+          'link': "/foo/anything",
+          'title': "Anything",
+        },
+      ]
 
-  describe "#popular_links_for_slug" do
-    it "adds GA tracking data to links" do
       expected_links = [
         {
-          text: "HMRC online services: sign in or set up an account",
-          href: "/log-in-register-hmrc-online-services",
+          text: "Policy on World Locations",
+          href: "/foo/policy_paper",
           data_attributes: {
             module: "ga4-link-tracker",
             ga4_track_links_only: "",
@@ -23,14 +28,14 @@
               type: "action",
               index_link: 1,
               index_total: 3,
-              text: "HMRC online services: sign in or set up an account",
+              text: "Policy on World Locations",
               section: "Popular tasks",
             },
           },
         },
         {
-          text: "Get information about a company",
-          href: "/get-information-about-a-company",
+          text: "PM attends summit on world location news pages",
+          href: "/foo/news_story",
           data_attributes: {
             module: "ga4-link-tracker",
             ga4_track_links_only: "",
@@ -39,14 +44,14 @@
               type: "action",
               index_link: 2,
               index_total: 3,
-              text: "Get information about a company",
+              text: "PM attends summit on world location news pages",
               section: "Popular tasks",
             },
           },
         },
         {
-          text: "Find an energy certificate",
-          href: "/find-energy-certificate",
+          text: "Anything",
+          href: "/foo/anything",
           data_attributes: {
             module: "ga4-link-tracker",
             ga4_track_links_only: "",
@@ -55,13 +60,13 @@
               type: "action",
               index_link: 3,
               index_total: 3,
-              text: "Find an energy certificate",
+              text: "Anything",
               section: "Popular tasks",
             },
           },
         },
       ]
-      expect(helper.popular_links_for_slug("business")).to eq(expected_links)
+      expect(helper.with_tracking_data(popular_content_from_search)).to eq(expected_links)
     end
   end
 end

spec/support/search_api_helpers.rb

@@ -253,6 +253,32 @@ def search_api_has_documents_for_browse_page(browse_page_content_id, document_sl
     end
   end
 
+  def search_api_has_popular_documents_for_level_one_browse(browse_content_item)
+    content_item = browse_content_item.with_indifferent_access
+    return if content_item["links"]["second_level_browse_pages"].blank?
+
+    level_two_browse_content_ids =
+      content_item["links"]["second_level_browse_pages"].map do |link|
+        link["content_id"]
+      end
+
+    fields = SearchApiFields::POPULAR_BROWSE_SEARCH_FIELDS
+
+    params = {
+      count: "3",
+      filter_any_mainstream_browse_page_content_ids: webmock_match_array(level_two_browse_content_ids),
+      order: "-popularity",
+      fields: webmock_match_array(fields),
+    }
+
+    results = %w[anything anything_else].map do |slug|
+      search_api_document_for_slug(slug, 1.hour.ago, "guide")
+    end
+
+    body = { results: }
+    stub_search(params:, body:)
+  end
+
   def section_tagged_content_list(doc_type, count = 1)
     content_list = []