Skip to content

Commit 271b60e

Browse files
Fix test
Signed-off-by: Ilya Dmitrichenko <[email protected]>
1 parent df1734a commit 271b60e

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

internal/controller/ocirepository_controller.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -408,7 +408,7 @@ func (r *OCIRepositoryReconciler) reconcileSource(ctx context.Context, sp *patch
408408
// Mark observations about the revision on the object
409409
defer func() {
410410
if !obj.GetArtifact().HasRevision(revision) {
411-
message := fmt.Sprintf("new revision '%s' for '%s'", revision, ref)
411+
message := fmt.Sprintf("new revision '%s' for '%s'", revision, obj.Spec.URL)
412412
if obj.GetArtifact() != nil {
413413
conditions.MarkTrue(obj, sourcev1.ArtifactOutdatedCondition, "NewRevision", message)
414414
}

internal/controller/ocirepository_controller_test.go

+3-2
Original file line numberDiff line numberDiff line change
@@ -1575,12 +1575,13 @@ func TestOCIRepository_reconcileSource_verifyOCISourceSignature_keyless(t *testi
15751575
Reference: tt.reference,
15761576
},
15771577
}
1578-
url := strings.TrimPrefix(obj.Spec.URL, "oci://") + ":" + tt.reference.Tag
1578+
1579+
url := strings.TrimPrefix(obj.Spec.URL, "oci://")
15791580

15801581
assertConditions := tt.assertConditions
15811582
for k := range assertConditions {
15821583
assertConditions[k].Message = strings.ReplaceAll(assertConditions[k].Message, "<revision>", tt.revision)
1583-
assertConditions[k].Message = strings.ReplaceAll(assertConditions[k].Message, "<url>", url)
1584+
assertConditions[k].Message = strings.ReplaceAll(assertConditions[k].Message, "<url>", obj.Spec.URL)
15841585
assertConditions[k].Message = strings.ReplaceAll(assertConditions[k].Message, "<provider>", "cosign")
15851586
}
15861587

0 commit comments

Comments
 (0)