googleapis / googleapis/google-cloud-python

google-auth: Clean up test coverage and outdated docstrings for mTLS and Identity Pool

Ouverte
#17,758 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
auth effort: low priority: p3 testing type: cleanup
Langage dominant
Python
Étoiles
5.4k
Forks
1.8k
Merge moyen
2 j 23 h
PR mergées (30 j)
123

Description

There are a few minor gaps in test coverage and test documentation that should be cleaned up in the `google-auth` library:

- In `tests/transport/test_aio_mtls_helper.py`, the docstring and comments for `test_get_client_cert_and_key_callback_sync` reference a `TypeError` fallback that is no longer used (the code now uses `inspect.isawaitable()`).
- The `MdsMtlsAdapter.close()` method in `google/auth/compute_engine/_mtls.py` lacks test coverage.
- There are several `RefreshError` exception paths in `google/auth/identity_pool.py` (specifically around leaf certificate retrieval and parsing) that are currently missing test coverage.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par tests/transport/test_aio_mtls_helper.py et mettez à jour la docstring et les commentaires de test_get_client_cert_and_key_callback_sync. Examinez ensuite google/auth/compute_engine/_mtls.py et google/auth/identity_pool.py avec leurs tests existants, en couvrant MdsMtlsAdapter.close() et les chemins RefreshError indiqués. Exécutez les tests concernés et vérifiez que les références obsolètes à TypeError ont été supprimées et que les chemins manquants sont couverts.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
testing
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Active
Clarté
Plutôt claire
Accessibilité débutants
74/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.