googleapis / googleapis/google-cloud-cpp

Use a `MockRestClient` in `storage/tests/error_injection_integration_test`

Ouverte
#14,428 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
api: storage type: cleanup
Langage dominant
C++
Étoiles
659
Forks
462
Merge moyen
1 j 2 h
PR mergées (30 j)
89

Description

This test does some low-level black magic to force a specific type of socket error.

1. The storage library should not be testing this, if anywhere, this belongs in the `RestClient` tests,
2. The storage library should just mock `RestClient` and assume it works.

Even if we decided to "move" these tests: it seems better to assume libcurl works, and just returns the right error code. If we want to make sure our wrappers work when libcurl returns the error codes corresponding to socket errors then (as libcurl cannot be easily mocked) we can use the testbench to force the error type we want.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par lire storage/tests/error_injection_integration_test et identifiez la configuration des erreurs de socket de bas niveau. Remplacez cette dépendance par un MockRestClient, tout en laissant la couverture des codes d’erreur de RestClient à ses propres tests ou au testbench, selon le cas. Le travail est terminé lorsque le test de storage ne force plus directement les erreurs de socket et vérifie toujours la gestion des erreurs prévue.

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

Évaluation

Stack technique
cpp
Domaine
testing
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

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