Merge pull request #202 from DFE-Digital/show-results-count

Show results and handle empty result
DFE-Digital · Aug 23, 2024 · 94a426b · 94a426b
2 parents 6171aa6 + 83e7996
commit 94a426b
Show file tree

Hide file tree

Showing 3 changed files with 321 additions and 3 deletions.
diff --git a/app/lib/local_authority/search.rb b/app/lib/local_authority/search.rb
@@ -5,6 +5,8 @@ def initialize(scope, query)
   end
 
   def call
+    return [] if geo_result.nil?
+
     result = @scope.order_by_distance_from(search_centre)
 
     if search_polygon

diff --git a/app/views/search/results.html.erb b/app/views/search/results.html.erb
@@ -8,6 +8,20 @@
 </div>
 
 <div class="govuk-grid-row">
+  <div class="govuk-grid-column-two-thirds">
+    <p class="govuk-body-l">
+      <b><%= pluralize(@search.hubs.count, 'result') %> found</b>
+    </p>
+    <% if @search.hubs.count.zero? %>
+      <p class="govuk-body">
+        Check that the location you have entered is in England and:
+        <ul class="govuk-list govuk-list--bullet">
+          <li>a valid first part of a postcode</li>
+          <li>or, a town or city</li>
+        </ul>
+      </p>
+    <% end %>
+  </div>
   <div class="govuk-grid-column-two-thirds">
     <p>
       <%= link_to 'Change location', { action: :form } %>

diff --git a/spec/fixtures/vcr_cassettes/geocoder.yml b/spec/fixtures/vcr_cassettes/geocoder.yml