google/adk-go

Missing contract test coverage for GetArtifactVersion()

Open

#681 aperta il 26 mar 2026

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Go (699 fork)github user discovery
good first issue

Metriche repository

Star
 (8108 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor