Small improvements, make address search and existing search more similar

app/controllers/address_search_controller.rb

@@ -14,7 +14,7 @@ class AddressSearchController < ApplicationController
         render json: { error: service.error }, status: :unprocessable_entity
       else
         presenter = UprnDataPresenter.new(service.result)
-        render json: [{ address: presenter.address, uprn: presenter.uprn }]
+        render json: [{ text: presenter.address, value: presenter.uprn }]
       end
     elsif query.match?(/[a-zA-Z]/)
       # Query contains letters, assume it's an address
@@ -26,7 +26,7 @@ class AddressSearchController < ApplicationController
       else
         results = service.result.map do |result|
           presenter = AddressDataPresenter.new(result)
-          { address: presenter.address, uprn: presenter.uprn }
+          { text: presenter.address, value: presenter.uprn }
         end
         render json: results
       end
@@ -45,7 +45,7 @@ class AddressSearchController < ApplicationController
       else
         formatted_results = results.map do |result|
           presenter = AddressDataPresenter.new(result)
-          { address: presenter.address, uprn: presenter.uprn }
+          { text: presenter.address, value: presenter.uprn }
         end
         render json: formatted_results
       end

app/frontend/controllers/address_search_controller.js

@@ -13,7 +13,7 @@ const fetchAndPopulateSearchResults = async (query, populateResults, searchUrl,
   if (/\S/.test(query)) {
     const results = await fetchOptions(query, searchUrl)
     populateOptions(results, selectEl)
-    populateResults(Object.values(results).map((o) => o.address))
+    populateResults(Object.values(results).map((o) => o.text))
   }
 }
 
@@ -22,9 +22,8 @@ const populateOptions = (results, selectEl) => {
 
   results.forEach((result) => {
     const option = document.createElement('option')
-    option.value = result.uprn
+    option.value = result.value
-    option.innerHTML = result.address
+    option.innerHTML = result.text
-    option.setAttribute('address', result.address)
     selectEl.appendChild(option)
     options.push(option)
   })