Image size must be set

Adding a test

Signed-off-by: Silvin Lubecki <[email protected]>

converter/convert.go

@@ -225,11 +225,14 @@ func makeDescriptor(baseImage bundle.BaseImage, targetReference reference.Named,
 	if err != nil {
 		return ocischemav1.Descriptor{}, err
 	}
+	if baseImage.Size == 0 {
+		return ocischemav1.Descriptor{}, fmt.Errorf("image %q size is not set", relocatedImage)
+	}
 
 	return ocischemav1.Descriptor{
 		Digest:    digested.Digest(),
 		MediaType: mediaType,
-		Size:      int64(baseImage.Size), // TODO: mutate the bundle if the size is not set
+		Size:      int64(baseImage.Size),
 	}, nil
 }
 

converter/convert_test.go

@@ -82,6 +82,12 @@ func TestConvertFromFixedUpBundleToOCI(t *testing.T) {
 	assert.ErrorContains(t, err, "invalid invocation image: "+
 		"image \"Some/iNvalid/Ref\" is not a valid image reference: invalid reference format: repository name must be lowercase")
 
+	// Invalid size
+	src = tests.MakeTestBundle()
+	src.InvocationImages[0].Size = 0
+	_, err = ConvertBundleToOCIIndex(src, named, bundleConfigDescriptor, relocationMap)
+	assert.ErrorContains(t, err, "size is not set")
+
 	// mediatype ociindex
 	src = tests.MakeTestBundle()
 	src.InvocationImages[0].MediaType = ocischemav1.MediaTypeImageIndex