base_adresse: handle only-space q (#69889)
This commit is contained in:
parent
f3625c6b5d
commit
7d87e97137
|
@ -135,7 +135,7 @@ class BaseAdresse(AddressResource):
|
|||
return {'data': []}
|
||||
|
||||
q = q.strip()
|
||||
if not q[0].isalnum() or len(q) < 3:
|
||||
if not q or not q[0].isalnum() or len(q) < 3:
|
||||
return {'data': []}
|
||||
|
||||
try:
|
||||
|
|
|
@ -245,16 +245,22 @@ def test_base_adresse_search_qs_lat_lon(mocked_get, app, base_adresse):
|
|||
def test_base_adresse_search_qs_empty(app, base_adresse, mock_api_adresse_data_gouv_fr_search):
|
||||
resp = app.get('/base-adresse/%s/search?q=' % base_adresse.slug)
|
||||
assert len(resp.json) == 0
|
||||
resp = app.get('/base-adresse/%s/search?q= ' % base_adresse.slug)
|
||||
assert len(resp.json) == 0
|
||||
|
||||
|
||||
def test_base_adresse_search_qs_not_alphanumeric(app, base_adresse, mock_api_adresse_data_gouv_fr_search):
|
||||
resp = app.get('/base-adresse/%s/search?q=**notalphanumeric' % base_adresse.slug)
|
||||
assert len(resp.json) == 0
|
||||
resp = app.get('/base-adresse/%s/search?q= **notalpha ' % base_adresse.slug)
|
||||
assert len(resp.json) == 0
|
||||
|
||||
|
||||
def test_base_adresse_search_qs_too_short(app, base_adresse, mock_api_adresse_data_gouv_fr_search):
|
||||
resp = app.get('/base-adresse/%s/search?q=12' % base_adresse.slug)
|
||||
assert len(resp.json) == 0
|
||||
resp = app.get('/base-adresse/%s/search?q= ab ' % base_adresse.slug)
|
||||
assert len(resp.json) == 0
|
||||
|
||||
|
||||
def test_base_adresse_search_qs_parameters_error(app, base_adresse, mock_api_adresse_data_gouv_fr_search):
|
||||
|
|
Loading…
Reference in New Issue