users search api

misago/search/api.py

@@ -28,7 +28,7 @@ def search(request, search_provider=None):
             'url': reverse('misago:search', kwargs={'search_provider': provider.url}),
             'api': reverse('misago:api:search', kwargs={'search_provider': provider.url}),
             'results': None,
-            'time': 0,
+            'time': None,
         }
 
         if not search_provider or search_provider == provider.url:

misago/search/tests/test_api.py

@@ -0,0 +1,57 @@
+from django.urls import reverse
+from django.utils import six
+
+from misago.acl.testutils import override_acl
+from misago.users.testutils import AuthenticatedUserTestCase
+
+from ..searchproviders import searchproviders
+
+
+class SearchApiTests(AuthenticatedUserTestCase):
+    def setUp(self):
+        super(SearchApiTests, self).setUp()
+
+        self.test_link = reverse('misago:api:search')
+
+    def test_no_permission(self):
+        """api validates permission to search"""
+        override_acl(self.user, {
+            'can_search': 0
+        })
+
+        response = self.client.get(self.test_link)
+
+        self.assertContains(
+            response, "have permission to search site", status_code=403)
+
+    def test_no_phrase(self):
+        """api handles no search query"""
+        response = self.client.get(self.test_link)
+        self.assertEqual(response.status_code, 200)
+
+        providers = searchproviders.get_providers(True)
+        for i, provider in enumerate(response.json()):
+            provider_api = reverse('misago:api:search', kwargs={
+                'search_provider': providers[i].url
+            })
+            self.assertEqual(provider_api, provider['api'])
+
+            self.assertEqual(six.text_type(providers[i].name), provider['name'])
+            self.assertEqual(provider['results']['results'], [])
+            self.assertEqual(int(provider['time']), 0)
+
+    def test_empty_(self):
+        """api handles empty search query"""
+        response = self.client.get('%s?q=' % self.test_link)
+        self.assertEqual(response.status_code, 200)
+
+        providers = searchproviders.get_providers(True)
+        for i, provider in enumerate(response.json()):
+            provider_api = reverse('misago:api:search', kwargs={
+                'search_provider': providers[i].url
+            })
+            self.assertEqual(provider_api, provider['api'])
+
+            self.assertEqual(six.text_type(providers[i].name), provider['name'])
+            self.assertEqual(provider['results']['results'], [])
+            self.assertEqual(int(provider['time']), 0)

misago/users/tests/test_search.py

@@ -0,0 +1,108 @@
+from django.urls import reverse
+
+from misago.acl.testutils import override_acl
+from misago.users.testutils import AuthenticatedUserTestCase
+
+
+class SearchApiTests(AuthenticatedUserTestCase):
+    def setUp(self):
+        super(SearchApiTests, self).setUp()
+
+        self.api_link = reverse('misago:api:search')
+
+    def test_no_permission(self):
+        """api respects permission to search users"""
+        override_acl(self.user, {
+            'can_search_users': 0
+        })
+
+        response = self.client.get(self.api_link)
+        self.assertEqual(response.status_code, 200)
+        self.assertNotIn('users', [p['id'] for p in response.json()])
+
+    def test_no_query(self):
+        """api handles no search query"""
+        response = self.client.get(self.api_link)
+        self.assertEqual(response.status_code, 200)
+
+        reponse_json = response.json()
+        self.assertIn('users', [p['id'] for p in reponse_json])
+
+        for provider in reponse_json:
+            if provider['id'] == 'users':
+                self.assertEqual(provider['results']['results'], [])
+
+    def test_empty_query(self):
+        """api handles empty search query"""
+        response = self.client.get('%s?q=' % self.api_link)
+        self.assertEqual(response.status_code, 200)
+
+        reponse_json = response.json()
+        self.assertIn('users', [p['id'] for p in reponse_json])
+
+        for provider in reponse_json:
+            if provider['id'] == 'users':
+                self.assertEqual(provider['results']['results'], [])
+
+    def test_short_query(self):
+        """api handles short search query"""
+        response = self.client.get('%s?q=%s' % (self.api_link, self.user.username[0]))
+        self.assertEqual(response.status_code, 200)
+
+        reponse_json = response.json()
+        self.assertIn('users', [p['id'] for p in reponse_json])
+
+        for provider in reponse_json:
+            if provider['id'] == 'users':
+                results = provider['results']['results']
+                self.assertEqual(len(results), 1)
+                self.assertEqual(results[0]['id'], self.user.id)
+
+    def test_exact_match(self):
+        """api handles exact search query"""
+        response = self.client.get('%s?q=%s' % (self.api_link, self.user.username))
+        self.assertEqual(response.status_code, 200)
+
+        reponse_json = response.json()
+        self.assertIn('users', [p['id'] for p in reponse_json])
+
+        for provider in reponse_json:
+            if provider['id'] == 'users':
+                results = provider['results']['results']
+                self.assertEqual(len(results), 1)
+                self.assertEqual(results[0]['id'], self.user.id)
+
+    def test_tail_match(self):
+        """api handles last three chars match query"""
+        response = self.client.get('%s?q=%s' % (self.api_link, self.user.username[-3:]))
+        self.assertEqual(response.status_code, 200)
+
+        reponse_json = response.json()
+        self.assertIn('users', [p['id'] for p in reponse_json])
+
+        for provider in reponse_json:
+            if provider['id'] == 'users':
+                results = provider['results']['results']
+                self.assertEqual(len(results), 1)
+                self.assertEqual(results[0]['id'], self.user.id)
+
+    def test_no_match(self):
+        """api handles no match"""
+        response = self.client.get('%s?q=BobBoberson' % self.api_link)
+        self.assertEqual(response.status_code, 200)
+
+        reponse_json = response.json()
+        self.assertIn('users', [p['id'] for p in reponse_json])
+
+        for provider in reponse_json:
+            if provider['id'] == 'users':
+                self.assertEqual(provider['results']['results'], [])
+
+
+class SearchProviderApiTests(SearchApiTests):
+    def setUp(self):
+        super(SearchProviderApiTests, self).setUp()
+
+        self.api_link = reverse('misago:api:search', kwargs={
+            'search_provider': 'users'
+        })