googleapis / googleapis/google-cloud-python
google-auth: Clean up test coverage and outdated docstrings for mTLS and Identity Pool
- 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