google/adk-go

Missing contract test coverage for GetArtifactVersion()

Open

#681 ouverte le 26 mars 2026

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Go (699 forks)github user discovery
good first issue

Métriques du dépôt

Stars
 (8 108 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

The new func (s *gcsService) GetArtifactVersion(ctx context.Context, req *artifact.GetArtifactVersionRequest) method does not appear to be covered by the contract tests yet.

https://github.com/google/adk-go/blob/b8eb8c5f07cf43889a02bce2eac2d104a81fb927/artifact/request_validation_test.go#L329

It would be useful for the contract tests to define which ArtifactVersion fields are required across backends, which are optional, and whether backend-specific omissions are allowed.

Guide contributeur

Direction de recherche
Examinez les tests de contrat existants dans artifact/request validation test.go pour comprendre le modèle, puis ajoutez la couverture pour GetArtifactVersion, en définissant les champs obligatoires et facultatifs.
Stack technique
go
Domaine
backendapi
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.
Plutôt claire
Prérequis
Gotesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50