Skip to content

Commit 9f75d1a

Browse files
committed
Fixes #102: Improve aggregations method tests.
This commits adds: - Proper response message on null query arguments. - Tests for aggregations() method with null query arguments.
1 parent e74865b commit 9f75d1a

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

loklak.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -420,7 +420,7 @@ def aggregations(self, query=None, since=None, until=None,
420420
return json.dumps(return_to_user)
421421
else:
422422
return_to_user = {}
423-
return_to_user['error'] = ('No Query string has been given to run'
423+
return_to_user['error'] = ('No Query string has been given to run '
424424
'query for aggregations')
425425
return json.dumps(return_to_user)
426426

tests/test.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,9 @@ def test_aggregations(self):
127127
data = result.json()
128128
self.assertEqual(result.status_code, 200)
129129
self.assertTrue('aggregations' in data)
130+
131+
result = self.loklak.aggregations()
132+
self.assertEqual(result, '{"error": "No Query string has been given to run query for aggregations"}')
130133
# self.assertTrue('hashtags' in data['aggregations'])
131134
# self.assertTrue('mentions' in data['aggregations'])
132135

0 commit comments

Comments
 (0)