Add a comment justifying lack of unit test coverage

mtrmac · mtrmac · commit 0cb521923172 · 2025-01-06T18:01:16.000+01:00
... to reinforce the rule that *_transport.go needs to have comprehensive
unit tests, to the fullest possible extent.

Signed-off-by: Miloslav Trmač &lt;mitr@redhat.com&gt;
diff --git a/oci/layout/oci_transport.go b/oci/layout/oci_transport.go
@@ -219,7 +219,7 @@ func (ref ociReference) getManifestDescriptor() (imgspecv1.Descriptor, int, erro
 	}
 
 	switch {
-	case ref.image != "" && ref.sourceIndex != -1:
+	case ref.image != "" && ref.sourceIndex != -1: // Coverage: newReference refuses to create such references.
 		return imgspecv1.Descriptor{}, -1, fmt.Errorf("Internal error: Cannot have both ref %s and source index @%d",
 			ref.image, ref.sourceIndex)
 

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@ func (ref ociReference) getManifestDescriptor() (imgspecv1.Descriptor, int, erro`
`219`	`219`	`}`
`220`	`220`
`221`	`221`	`switch {`
`222`		`- case ref.image != "" && ref.sourceIndex != -1:`
	`222`	`+ case ref.image != "" && ref.sourceIndex != -1: // Coverage: newReference refuses to create such references.`
`223`	`223`	`return imgspecv1.Descriptor{}, -1, fmt.Errorf("Internal error: Cannot have both ref %s and source index @%d",`
`224`	`224`	`ref.image, ref.sourceIndex)`
`225`	`225`