-
Notifications
You must be signed in to change notification settings - Fork 619
Missing contract test coverage for GetArtifactVersion() #681
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers
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.
adk-go/artifact/request_validation_test.go
Line 329 in b8eb8c5
| func TestGetArtifactVersionRequest_Validate(t *testing.T) { |
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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers