Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

Commit 58c20dc

Browse files
committed
Used sort for search tests
The results need to be in a guaranteed order to be able to assert they are the same as expected, that means we need a sort or group_sort.
1 parent b7f2c76 commit 58c20dc

File tree

1 file changed

+21
-19
lines changed

1 file changed

+21
-19
lines changed

tests/unit/database_tests.py

Lines changed: 21 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1278,6 +1278,7 @@ def test_get_search_result_executes_search_query(self):
12781278
'searchddoc001',
12791279
'searchindex001',
12801280
query='julia*',
1281+
sort='_id<string>',
12811282
limit=5,
12821283
include_docs=True
12831284
)
@@ -1292,23 +1293,23 @@ def test_get_search_result_executes_search_query(self):
12921293
{'rows': [{'fields': {'name': 'julia'}, 'doc': {'_id': 'julia000',
12931294
'age': 0,
12941295
'name': 'julia'},
1295-
'id': 'julia000', 'order': [1.0, 0]},
1296+
'id': 'julia000', 'order': ['julia000', 0]},
12961297
{'fields': {'name': 'julia'}, 'doc': {'_id': 'julia001',
12971298
'age': 1,
12981299
'name': 'julia'},
1299-
'id': 'julia001', 'order': [1.0, 0]},
1300+
'id': 'julia001', 'order': ['julia001', 0]},
13001301
{'fields': {'name': 'julia'},'doc': {'_id': 'julia002',
13011302
'age': 2,
13021303
'name': 'julia'},
1303-
'id': 'julia002', 'order': [1.0, 0]},
1304-
{'fields': {'name': 'julia'}, 'doc': {'_id': 'julia004',
1305-
'age': 4,
1304+
'id': 'julia002', 'order': ['julia002', 0]},
1305+
{'fields': {'name': 'julia'}, 'doc': {'_id': 'julia003',
1306+
'age': 3,
13061307
'name': 'julia'},
1307-
'id': 'julia004', 'order': [1.0, 1]},
1308+
'id': 'julia003', 'order': ['julia003', 0]},
13081309
{'fields': {'name': 'julia'},
1309-
'doc': {'_id': 'julia005', 'age': 5,
1310+
'doc': {'_id': 'julia004', 'age': 4,
13101311
'name': 'julia'},
1311-
'id': 'julia005', 'order': [1.0, 1]}], 'total_rows': 100}
1312+
'id': 'julia004', 'order': ['julia004', 1]}], 'total_rows': 100}
13121313
)
13131314

13141315
def test_get_search_result_executes_search_query_with_group_option(self):
@@ -1322,7 +1323,8 @@ def test_get_search_result_executes_search_query_with_group_option(self):
13221323
'searchindex001',
13231324
query='name:julia*',
13241325
group_field='_id',
1325-
group_limit=5
1326+
group_limit=5,
1327+
group_sort='_id<string>'
13261328
)
13271329
# for group parameter options, 'rows' results are within 'groups' key
13281330
self.assertEqual(len(resp['groups']), 5)
@@ -1332,18 +1334,18 @@ def test_get_search_result_executes_search_query_with_group_option(self):
13321334
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia000',
13331335
'order': [1.0, 0]}], 'total_rows': 1,
13341336
'by': 'julia000'},
1337+
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia001',
1338+
'order': [1.0, 0]}], 'total_rows': 1,
1339+
'by': 'julia001'},
1340+
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia002',
1341+
'order': [1.0, 0]}], 'total_rows': 1,
1342+
'by': 'julia002'},
1343+
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia003',
1344+
'order': [1.0, 0]}], 'total_rows': 1,
1345+
'by': 'julia003'},
13351346
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia004',
13361347
'order': [1.0, 1]}], 'total_rows': 1,
1337-
'by': 'julia004'},
1338-
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia008',
1339-
'order': [1.0, 2]}], 'total_rows': 1,
1340-
'by': 'julia008'},
1341-
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia010',
1342-
'order': [1.0, 3]}], 'total_rows': 1,
1343-
'by': 'julia010'},
1344-
{'rows': [{'fields': {'name': 'julia'}, 'id': 'julia014',
1345-
'order': [1.0, 4]}], 'total_rows': 1,
1346-
'by': 'julia014'}
1348+
'by': 'julia004'}
13471349
]}
13481350
)
13491351

0 commit comments

Comments
 (0)