Project

General

Profile

Development #51368

un POST sur /api/users/?get_or_create=email considère même les comptes désactivés

Added by Thomas Noël 2 days ago. Updated 1 day ago.

Status:
Solution validée
Priority:
Normal
Category:
-
Target version:
-
Start date:
23 Feb 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Il faudrait que le get du get_or_create ne prenne pas en considération les comptes deactivation/deleted


Files

History

#2

Updated by Thomas Noël 2 days ago

Voici, selon moi, le test qui devrait passer :

diff --git a/tests/test_api.py b/tests/test_api.py
index cf276c6e..5b13d81b 100644
--- a/tests/test_api.py
+++ b/tests/test_api.py
@@ -1462,6 +1462,11 @@ def test_api_users_get_or_create(settings, app, admin):
     assert User.objects.get(id=id).password != password
     assert User.objects.get(id=id).check_password('secret')

+    # do not get deleted user, create a new one
+    User.objects.get(id=id).mark_as_deleted()
+    resp = app.post_json('/api/users/?get_or_create=email', params=payload, status=200)
+    assert id != resp.json['id']
+

 def test_api_users_get_or_create_email_is_unique(settings, app, admin):
     settings.A2_EMAIL_IS_UNIQUE = True
#3

Updated by Benjamin Dauvergne 2 days ago

  • Assignee set to Benjamin Dauvergne
#5

Updated by Paul Marillonnet 1 day ago

  • Status changed from Solution proposée to Solution validée

Le build est cassé mais rien à voir avec ce ticket, je vais faire un autre ticket.
Sinon c’est bon pour moi.

#6

Updated by Paul Marillonnet 1 day ago

Paul Marillonnet a écrit :

Le build est cassé mais rien à voir avec ce ticket, je vais faire un autre ticket.

#51376

Also available in: Atom PDF