Mopidy: Déclaration d'assertion utilisée en dehors des tests

J'ai marqué cela comme un bon premier numéro. La solution immédiate est de passer de :

assert condition, "Error message"

À:

if not condition:
    raise AssertionError("error message")

Cela fait que le code se comporte exactement de la même manière qu'aujourd'hui, mais ne sera pas optimisé lors de l'exécution de Python avec -O .

@jodal , dois-je résoudre ce problème ou laisser d'autres contributeurs pour la première fois le résoudre ?

De plus, si vous voulez que les étudiants de première année le résolvent, j'ai besoin d'une suggestion de votre part, c'est-à-dire, et si nous divisons ce problème en plusieurs autres problèmes ? Je veux dire casser le problème en fonction des fichiers.

N'hésitez pas à vous en occuper si cela vous intéresse !

S'il s'agissait de très nombreuses instances du problème, il pourrait être judicieux de le diviser en plusieurs demandes d'extraction par zone de la base de code. Cependant, il semble qu'il n'y ait pas beaucoup de cas, donc je suggère de faire un seul PR corrigeant tous les cas.

Compte de assert par fichier, hors tests :

$ rg -c assert mopidy
mopidy/ext.py:3
mopidy/core/actor.py:1
mopidy/core/tracklist.py:8
mopidy/audio/utils.py:1
mopidy/core/playback.py:1

Hey @pnijhara @jodal Je veux travailler sur ce problème. Pouvez-vous s'il vous plaît m'attribuer ceci car je suis le premier contributeur.

Il y a des informations sur https://docs.mopidy.com/en/latest/contributing/ et aussi sur https://docs.mopidy.com/en/latest/devenv/ sur la façon de configurer un environnement de développement et d'exécuter les tests et linters etc.