Merge pull request #2837 from betagouv/refactor-ban

Refactor ban

app/controllers/address_controller.rb

@@ -1,5 +1,5 @@
-class Ban::SearchController < ApplicationController
-  def get
+class AddressController < ApplicationController
+  def suggestions
     request = params[:request]
 
     json = ApiAdresse::AddressAdapter.new(request).get_suggestions.map do |value|
@@ -9,7 +9,7 @@ class Ban::SearchController < ApplicationController
     render json: json
   end
 
-  def get_address_point
+  def geocode
     request = params[:request]
 
     point = ApiAdresse::PointAdapter.new(request).geocode

app/javascript/old_design/carto.js

@@ -63,7 +63,7 @@ function addEventFreeDraw(freeDraw) {
 }
 
 function getAddressPoint(map, request) {
-  getJSON('/ban/address_point', { request }).then(data => {
+  getJSON('/address/geocode', { request }).then(data => {
     if (data.lat !== null) {
       map.setView(new L.LatLng(data.lat, data.lon), data.zoom);
     }

app/javascript/shared/autocomplete.js

@@ -4,7 +4,7 @@ import { getJSON, fire } from '@utils';
 const sources = [
   {
     type: 'address',
-    url: '/ban/search'
+    url: '/address/suggestions'
   },
   {
     type: 'path',

config/routes.rb

@@ -250,10 +250,8 @@ Rails.application.routes.draw do
   # Addresses
   #
 
-  namespace :ban do
-    get 'search' => 'search#get'
-    get 'address_point' => 'search#get_address_point'
-  end
+  get 'address/suggestions' => 'address#suggestions'
+  get 'address/geocode' => 'address#geocode'
 
   namespace :invites do
     post 'dossier/:dossier_id' => '/invites#create', as: 'dossier'

spec/controllers/address_controller_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
-describe Ban::SearchController, type: :controller do
-  describe '#GET get' do
-    subject { get :get, params: { request: request } }
+describe AddressController, type: :controller do
+  describe '#GET suggestions' do
+    subject { get :suggestions, params: { request: request } }
 
     before do
       subject
@@ -23,9 +23,9 @@ describe Ban::SearchController, type: :controller do
     end
   end
 
-  describe '#GET get_address_point' do
+  describe '#GET geocode' do
     let(:dossier_id) { "1" }
-    subject { get :get_address_point, params: { request: request, dossier_id: dossier_id } }
+    subject { get :geocode, params: { request: request, dossier_id: dossier_id } }
 
     before do
       subject