Commit 6b39341e authored by Markus Scheidgen's avatar Markus Scheidgen
Browse files

Fixed search test to use comma separated atoms instead of not supported multiple parameter usage.

parent 02a33055
Pipeline #47272 failed with stages
in 28 minutes and 5 seconds
Subproject commit 6744335e87c6ec60190c092eedc4c12d54baae17
Subproject commit 9be589938071b2fd4f884f8d930d736fe32b6166
......@@ -244,6 +244,7 @@ def _construct_search(q: Q = None, time_range: Tuple[datetime, datetime] = None,
for item in values:
if key == 'atoms':
# TODO This should be configurable and not specific to atoms
items = item.split(',')
else:
items = [item]
......@@ -263,7 +264,7 @@ def scroll_search(
:func:`aggregate_search`, but pagination is replaced with scrolling, no ordering,
and no aggregation information is given.
Scrolling is done by calling this function again and again with the same ``scoll_id``.
Scrolling is done by calling this function again and again with the same ``scroll_id``.
Each time, this function will return the next batch of search results.
See see :func:`aggregate_search` for additional ``kwargs``
......
......@@ -675,11 +675,7 @@ class TestRepo():
(0, 'quantities', 'dos')
])
def test_search_quantities(self, client, example_elastic_calcs, no_warn, test_user_auth, calcs, quantity, value):
if isinstance(value, list):
query_string = '&'.join('%s=%s' % (quantity, item) for item in value)
else:
query_string = '%s=%s' % (quantity, value)
query_string = '%s=%s' % (quantity, ','.join(value) if isinstance(value, list) else value)
rv = client.get('/repo/?%s' % query_string, headers=test_user_auth)
assert rv.status_code == 200
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment