[damned-lies] Fixed api upload test after relaxing some rules



commit 61bf096e75cc63f17835f4465683819bff258940
Author: Claude Paroz <claude 2xlibre net>
Date:   Tue Nov 9 17:44:15 2021 +0100

    Fixed api upload test after relaxing some rules

 api/tests.py | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)
---
diff --git a/api/tests.py b/api/tests.py
index b9d215e9..482497ab 100644
--- a/api/tests.py
+++ b/api/tests.py
@@ -155,8 +155,13 @@ class APITests(TestCase):
             first_name='John', last_name='Translator',
             email='jt devnull com', username='translator'
         )
+        somebody = Person.objects.create(
+            email='some devnull com', username='somebody'
+        )
         team = Team.objects.get(name='fr')
         Role.objects.create(team=team, person=translator)
+        Role.objects.create(team=team, person=somebody)
+
         _, _, state = get_vertimus_state(
             Branch.objects.get(module__name='gnome-hello'),
             Domain.objects.get(module__name='gnome-hello', name='po'),
@@ -169,12 +174,15 @@ class APITests(TestCase):
             response = self.client.post(url, data={'file': File(fh)})
             self.assertRedirects(response, reverse('login') + f'?next={url}')
 
-        self.client.force_login(translator)
+        state.change_state(StateTranslating, person=translator)
+
+        # somebody cannot post translation if reserved by translator.
+        self.client.force_login(somebody)
         with test_po.open('rb') as fh:
             response = self.client.post(url, data={'file': File(fh)})
             self.assertEqual(response.status_code, 403)
 
-        state.change_state(StateTranslating, person=translator)
+        self.client.force_login(translator)
         with test_po.open('rb') as fh:
             response = self.client.post(url, data={'file': File(fh)})
         self.assertEqual(response.json(), {'result': 'OK'})


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]