Skip to content

Commit 0eef94f

Browse files
committed
Extend search tests
1 parent 41b2714 commit 0eef94f

File tree

7 files changed

+528
-30
lines changed

7 files changed

+528
-30
lines changed

tests/onegov/agency/test_views.py

+35-24
Original file line numberDiff line numberDiff line change
@@ -725,9 +725,21 @@ def test_excel_export_not_logged_in(client):
725725
@mark.flaky(reruns=3)
726726
def test_basic_search(client_with_es):
727727
client = client_with_es
728-
729728
client.login_admin()
729+
anom = client.spawn()
730+
731+
# basic test
732+
assert 'Resultate' in client.get('/search?q=test')
733+
assert client.get('/search/suggest?q=test').json == []
734+
assert 'Resultate' in anom.get('/search?q=test')
735+
assert anom.get('/search/suggest?q=test').json == []
730736

737+
assert 'Resultate' in client.get('/search-postgres?q=test')
738+
assert client.get('/search-postgres/suggest?q=test').json == []
739+
assert 'Resultate' in anom.get('/search-postgres?q=test')
740+
assert anom.get('/search-postgres/suggest?q=test').json == []
741+
742+
# Add data
731743
page = client.get('/settings').click("Organisationen", index=1)
732744
page.form['agency_phone_internal_digits'] = 4
733745
page.form.submit()
@@ -750,41 +762,40 @@ def test_basic_search(client_with_es):
750762
manage.form.submit()
751763

752764
client.app.es_client.indices.refresh(index='_all')
753-
client = client.spawn()
754765

755766
# Test search results elasticsearch
756-
assert 'Rivera' in client.get('/search?q=Nick')
757-
assert 'Nick' in client.get('/search?q=Rivera')
758-
assert 'Nick' in client.get('/search?q=Doctor')
759-
assert 'Nick' in client.get('/search?q=+12345678901')
760-
assert 'Nick' in client.get('/search?q=0345678901')
761-
assert 'Nick' in client.get('/search?q=8911')
762-
assert 'Hospital Springfield' in client.get('/search?q=Hospital')
763-
assert 'Nick' in client.get('/search?q=Anesthetist')
767+
assert 'Rivera' in anom.get('/search?q=Nick')
768+
assert 'Nick' in anom.get('/search?q=Rivera')
769+
assert 'Nick' in anom.get('/search?q=Doctor')
770+
assert 'Nick' in anom.get('/search?q=+12345678901')
771+
assert 'Nick' in anom.get('/search?q=0345678901')
772+
assert 'Nick' in anom.get('/search?q=8911')
773+
assert 'Hospital Springfield' in anom.get('/search?q=Hospital')
774+
assert 'Nick' in anom.get('/search?q=Anesthetist')
764775

765776
# Test suggestions
766-
assert 'Nick Rivera (Doctor)' in client.get('/search/suggest?q=Nic').json
767-
assert 'Rivera Nick (Doctor)' in client.get('/search/suggest?q=Riv').json
768-
assert '8911 Rivera Nick (Doctor)' in client.get(
777+
assert 'Nick Rivera (Doctor)' in anom.get('/search/suggest?q=Nic').json
778+
assert 'Rivera Nick (Doctor)' in anom.get('/search/suggest?q=Riv').json
779+
assert '8911 Rivera Nick (Doctor)' in anom.get(
769780
'/search/suggest?q=89').json
770781

771782
# postgres
772-
assert 'Rivera' in client.get('/search-postgres?q=Nick')
773-
assert 'Nick' in client.get('/search-postgres?q=Rivera')
774-
assert 'Nick' in client.get('/search-postgres?q=Doctor')
775-
assert 'Nick' in client.get('/search-postgres?q=345678901')
776-
assert 'Nick' in client.get('/search-postgres?q=345678911')
777-
assert 'Nick' in client.get('/search-postgres?q=8911')
778-
assert 'Hospital Springfield' in client.get('/search-postgres?q=Hospital')
779-
assert 'Nick' in client.get('/search-postgres?q=Anesthetist')
783+
assert 'Rivera' in anom.get('/search-postgres?q=Nick')
784+
assert 'Nick' in anom.get('/search-postgres?q=Rivera')
785+
assert 'Nick' in anom.get('/search-postgres?q=Doctor')
786+
assert 'Nick' in anom.get('/search-postgres?q=345678901')
787+
assert 'Nick' in anom.get('/search-postgres?q=345678911')
788+
assert 'Nick' in anom.get('/search-postgres?q=8911')
789+
assert 'Hospital Springfield' in anom.get('/search-postgres?q=Hospital')
790+
assert 'Nick' in anom.get('/search-postgres?q=Anesthetist')
780791

781792
# Test suggestions (no autocomplete)
782793
expected = ['Nick', 'Rivera', '(Doctor)'] # word order in json is changing
783-
assert all(v in client.get('/search-postgres/suggest?q=Nick').json[0]
794+
assert all(v in anom.get('/search-postgres/suggest?q=Nick').json[0]
784795
for v in expected)
785-
assert all(v in client.get('/search-postgres/suggest?q=Rivera').json[0]
796+
assert all(v in anom.get('/search-postgres/suggest?q=Rivera').json[0]
786797
for v in expected)
787-
assert all(v in client.get('/search-postgres/suggest?q=8911').json[0]
798+
assert all(v in anom.get('/search-postgres/suggest?q=8911').json[0]
788799
for v in expected)
789800

790801

tests/onegov/feriennet/test_views.py

+16
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,22 @@ def test_activity_communication(broadcast, authenticate, connect, client,
264264
assert "Using a Raspberry Pi we will learn Python" in message
265265

266266

267+
def test_basic_search(client_with_es):
268+
client = client_with_es
269+
anom = client.spawn()
270+
271+
# basic test
272+
assert 'Resultate' in client.get('/search?q=test')
273+
assert client.get('/search/suggest?q=test').json == []
274+
assert 'Resultate' in anom.get('/search?q=test')
275+
assert anom.get('/search/suggest?q=test').json == []
276+
277+
assert 'Resultate' in client.get('/search-postgres?q=test')
278+
assert client.get('/search-postgres/suggest?q=test').json == []
279+
assert 'Resultate' in anom.get('/search-postgres?q=test')
280+
assert anom.get('/search-postgres/suggest?q=test').json == []
281+
282+
267283
def test_activity_search(client_with_es, scenario):
268284
client = client_with_es
269285

tests/onegov/fsi/test_views_search.py

+7
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,11 @@ def test_basic_search(client_with_es):
66
anom = client.spawn()
77

88
assert 'Resultate' in client.get('/search?q=test')
9+
assert client.get('/search/suggest?q=test').json == []
910
anom.get('/search?q=test', status=403) # forbidden
11+
assert anom.get('/search/suggest?q=test', status=403) # forbidden
12+
13+
assert 'Resultate' in client.get('/search-postgres?q=test')
14+
assert client.get('/search/suggest?q=test').json == []
15+
anom.get('/search-postgres?q=test', status=403) # forbidden
16+
assert anom.get('/search/suggest?q=test', status=403) # forbidden

tests/onegov/org/test_views_search.py

+10-6
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
def test_basic_search(client_with_es):
2323
client = client_with_es
2424
client.login_admin()
25+
anom = client.spawn()
2526

2627
add_news = client.get('/news').click('Nachricht')
2728
add_news.form['title'] = "Now supporting fulltext search"
@@ -47,9 +48,9 @@ def test_basic_search(client_with_es):
4748

4849
# make sure anonymous doesn't see hidden things in the search results
4950
# elasticsearch
50-
assert "fulltext" in client.spawn().get('/search?q=fulltext')
51+
assert "fulltext" in anom.get('/search?q=fulltext')
5152
# postgres
52-
assert "fulltext" in client.spawn().get('/search-postgres?q=fulltext')
53+
assert "fulltext" in anom.get('/search-postgres?q=fulltext')
5354

5455
edit_news = news.click("Bearbeiten")
5556
edit_news.form['access'] = 'private'
@@ -58,14 +59,17 @@ def test_basic_search(client_with_es):
5859
client.app.es_client.indices.refresh(index='_all')
5960

6061
# elasticsearch
61-
assert ("Now supporting" not in
62-
client.spawn().get('/search?q=fulltext'))
62+
assert "Now supporting" not in anom.get('/search?q=fulltext')
63+
assert anom.get('/search/suggest?q=fulltext').json == []
6364
assert "Now supporting" in client.get('/search?q=fulltext')
65+
assert client.get('/search/suggest?q=fulltext').json == []
6466

6567
# postgres
66-
assert ("Now supporting" not in
67-
client.spawn().get('/search-postgres?q=fulltext'))
68+
assert "Now supporting" not in anom.get('/search-postgres?q=fulltext')
69+
assert anom.get('/search-postgres/suggest?q=fulltext').json == []
6870
assert "Now supporting" in client.get('/search-postgres?q=fulltext')
71+
assert (client.get('/search-postgres/suggest?q=fulltext').json == [
72+
'Now supporting fulltext search'])
6973

7074

7175
@pytest.mark.flaky(reruns=3)

0 commit comments

Comments
 (0)