Déplacement de code GPS

[tonial]

🤔 Pourquoi ?

On n'a pas très envie d'avoir du code utilisant les modèles de GPS dans users/models.py

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

• Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

[xavfernandez]

Je me dis qu'une petite fonction (interne au test):

def get_autocomplete_results(user):
    client.force_login(user)
    response = client.get(reverse("autocomplete:gps_users") + "?term=gps")
    return set(r["id"] for r in response.json()["results"])

# Employers should get the 3 job seekers.
employer = EmployerFactory(with_company=True)
assert get_autocomplete_results(employer) == {first_beneficiary.pk, second_beneficiary.pk, third_beneficiary.pk}

# Authorized prescribers should get the 3 job seekers.
prescriber = PrescriberOrganizationWithMembershipFactory(authorized=True).members.get()
assert get_autocomplete_results(prescriber) == {first_beneficiary.pk, second_beneficiary.pk, third_beneficiary.pk}

...

[francoisfreitag]

💯, ça me paraît également bien mieux.

Très bien de passer le test au niveau de la vue plutôt que le modèle.