Skip to content

Commit e5014cd

Browse files
Improve test assertion for ImageContent binary data
- Updated test to verify raw binary data matches expected value - Fixed comment to accurately describe what is being verified - Ensures test properly validates SDK's base64 encoding behavior Co-authored-by: SamMorrowDrums <4811358+SamMorrowDrums@users.noreply.github.com>
1 parent 7243045 commit e5014cd

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/github/repositories_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -568,8 +568,8 @@ func Test_GetFileContents_WithDisabledEmbeddedResourcesFlag(t *testing.T) {
568568
assert.Equal(t, tc.expectedMimeType, content.MIMEType)
569569
assert.NotNil(t, content.Meta)
570570
assert.NotNil(t, content.Annotations)
571-
// Verify data is base64 encoded
572-
assert.NotEmpty(t, content.Data)
571+
// Verify data contains raw binary (SDK handles base64 encoding during JSON marshaling)
572+
assert.Equal(t, mockBinaryContent, content.Data)
573573
}
574574
} else {
575575
// When flag is disabled, should use EmbeddedResource (default)

0 commit comments

Comments
 (0)