Merge pull request #10282 from tchak/fix-parse-ban-adresse
fix(adresse): fallback to city name if commune not found
This commit is contained in:
commit
37ae3142ff
2 changed files with 21 additions and 1 deletions
|
@ -101,7 +101,7 @@ class APIGeoService
|
|||
department_code:,
|
||||
region_name: region_name(region_code),
|
||||
region_code:,
|
||||
city_name: commune_name(department_code, city_code),
|
||||
city_name: commune_name(department_code, city_code) || properties['city'], # fallback to city name if commune not found
|
||||
city_code:
|
||||
}
|
||||
else
|
||||
|
|
|
@ -87,4 +87,24 @@ describe APIGeoService do
|
|||
expect(APIGeoService.epcis('01').first).to eq(code: '200042935', name: 'CA Haut - Bugey Agglomération')
|
||||
end
|
||||
end
|
||||
|
||||
describe 'parse_ban_address' do
|
||||
let(:features) { JSON.parse(Rails.root.join('spec/fixtures/files/api_address/address.json').read)['features'] }
|
||||
let(:feature) { features.first }
|
||||
subject { APIGeoService.parse_ban_address(feature) }
|
||||
|
||||
context 'with a valid code insee' do
|
||||
it { expect(subject[:city_name]).to eq('Paris') }
|
||||
end
|
||||
|
||||
context 'with an invalid code insee' do
|
||||
let(:feature) do
|
||||
features.first.tap {
|
||||
_1['properties']['citycode'] = '0000'
|
||||
}
|
||||
end
|
||||
|
||||
it { expect(subject[:city_name]).to eq('Paris') }
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue