openedx/openedx-platform

Test: Tests break when pycountries updates its list of countries

Ouverte

#37 611 ouverte le 5 nov. 2025

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Python (4 331 forks)auto 404
good first issuetesting

Métriques du dépôt

Stars
 (8 150 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Everytime pycountries updates the list of countries, edx-platform tests fail because we have a copy of the list from that library as a set of test constants: https://github.com/openedx/edx-platform/blob/master/openedx/core/djangoapps/user_api/tests/test_constants.py

This is super brittle and it's unclear what value it's providing. Do we need these tests? What are they doing? Can we test what we care about in a less brittle way?

Investigate the tests that use the SORTED_COUNTRIES constant and where it's used and see if we can get rid of it.

Guide contributeur

Direction de recherche
Examinez les tests qui utilisent la constante SORTED COUNTRIES, comprenez ce qu'ils testent et proposez une manière de tester la même fonctionnalité sans dépendre d'une copie de la liste pycountries.
Stack technique
python
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Pythonpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50