Skip to content

Commit 5035730

Browse files
committed
Set a critical annotation
Signed-off-by: twobiers <22715034+twobiers@users.noreply.github.com>
1 parent eb17cb3 commit 5035730

1 file changed

Lines changed: 26 additions & 6 deletions

File tree

pkg/reconciler/managed/api_test.go

Lines changed: 26 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -552,7 +552,7 @@ func TestRetryingCriticalAnnotationUpdater(t *testing.T) {
552552
}
553553

554554
setAnnotations := func(obj client.Object) error {
555-
obj.SetAnnotations(map[string]string{"getcalled": "true"})
555+
obj.SetAnnotations(map[string]string{"crossplane.io/external-name": "my-external-name"})
556556
return nil
557557
}
558558
objectReturnedByGet := &fake.LegacyManaged{}
@@ -574,7 +574,11 @@ func TestRetryingCriticalAnnotationUpdater(t *testing.T) {
574574
}, "abc", errBoom)),
575575
},
576576
args: args{
577-
o: &fake.LegacyManaged{},
577+
o: &fake.LegacyManaged{
578+
ObjectMeta: metav1.ObjectMeta{
579+
Annotations: map[string]string{"crossplane.io/external-name": "my-external-name"},
580+
},
581+
},
578582
},
579583
want: want{
580584
err: errors.Wrap(errBoom, errUpdateCriticalAnnotations),
@@ -588,11 +592,19 @@ func TestRetryingCriticalAnnotationUpdater(t *testing.T) {
588592
MockPatch: test.NewMockPatchFn(errBoom),
589593
},
590594
args: args{
591-
o: &fake.LegacyManaged{},
595+
o: &fake.LegacyManaged{
596+
ObjectMeta: metav1.ObjectMeta{
597+
Annotations: map[string]string{"crossplane.io/external-name": "my-external-name"},
598+
},
599+
},
592600
},
593601
want: want{
594602
err: errors.Wrap(errBoom, errUpdateCriticalAnnotations),
595-
o: &fake.LegacyManaged{},
603+
o: &fake.LegacyManaged{
604+
ObjectMeta: metav1.ObjectMeta{
605+
Annotations: map[string]string{"crossplane.io/external-name": "my-external-name"},
606+
},
607+
},
596608
},
597609
},
598610
"SuccessfulGetAfterAConflict": {
@@ -605,7 +617,11 @@ func TestRetryingCriticalAnnotationUpdater(t *testing.T) {
605617
}, "abc", errBoom)),
606618
},
607619
args: args{
608-
o: &fake.LegacyManaged{},
620+
o: &fake.LegacyManaged{
621+
ObjectMeta: metav1.ObjectMeta{
622+
Annotations: map[string]string{"crossplane.io/external-name": "my-external-name"},
623+
},
624+
},
609625
},
610626
want: want{
611627
err: errors.Wrap(kerrors.NewConflict(schema.GroupResource{
@@ -622,7 +638,11 @@ func TestRetryingCriticalAnnotationUpdater(t *testing.T) {
622638
MockPatch: test.NewMockPatchFn(errBoom),
623639
},
624640
args: args{
625-
o: &fake.LegacyManaged{},
641+
o: &fake.LegacyManaged{
642+
ObjectMeta: metav1.ObjectMeta{
643+
Annotations: map[string]string{"crossplane.io/external-name": "my-external-name"},
644+
},
645+
},
626646
},
627647
want: want{
628648
err: errors.Wrap(errBoom, errUpdateCriticalAnnotations),

0 commit comments

Comments
 (0)