Skip to content

Commit 9ff2621

Browse files
committed
test: Fix reconcile error expectations
Signed-off-by: Jatin Kumar <jatink.5251@gmail.com>
1 parent dbf0a41 commit 9ff2621

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

infra/feast-operator/internal/controller/featurestore_controller_test.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -460,9 +460,10 @@ var _ = Describe("FeatureStore Controller", func() {
460460

461461
cond = apimeta.FindStatusCondition(resource.Status.Conditions, feastdevv1.OnlineStoreReadyType)
462462
Expect(cond).ToNot(BeNil())
463-
Expect(cond.Status).To(Equal(metav1.ConditionTrue))
464-
Expect(cond.Reason).To(Equal(feastdevv1.ReadyReason))
463+
Expect(cond.Status).To(Equal(metav1.ConditionFalse))
464+
Expect(cond.Reason).To(Equal(feastdevv1.OnlineStoreFailedReason))
465465
Expect(cond.Type).To(Equal(feastdevv1.OnlineStoreReadyType))
466+
Expect(cond.Message).To(Equal("Error: Object " + resource.Namespace + "/" + deploy.Name + " is already owned by another Service controller " + name))
466467

467468
cond = apimeta.FindStatusCondition(resource.Status.Conditions, feastdevv1.ClientReadyType)
468469
Expect(cond).ToNot(BeNil())
@@ -1405,10 +1406,10 @@ var _ = Describe("FeatureStore Controller", func() {
14051406

14061407
cond = apimeta.FindStatusCondition(resource.Status.Conditions, feastdevv1.OnlineStoreReadyType)
14071408
Expect(cond).ToNot(BeNil())
1408-
Expect(cond.Status).To(Equal(metav1.ConditionTrue))
1409-
Expect(cond.Reason).To(Equal(feastdevv1.ReadyReason))
1409+
Expect(cond.Status).To(Equal(metav1.ConditionFalse))
1410+
Expect(cond.Reason).To(Equal(feastdevv1.OnlineStoreFailedReason))
14101411
Expect(cond.Type).To(Equal(feastdevv1.OnlineStoreReadyType))
1411-
Expect(cond.Message).To(Equal(feastdevv1.OnlineStoreReadyMessage))
1412+
Expect(cond.Message).To(Equal("Error: Object " + resource.Namespace + "/" + deploy.Name + " is already owned by another Service controller " + name))
14121413

14131414
Expect(resource.Status.Phase).To(Equal(feastdevv1.FailedPhase))
14141415
})

0 commit comments

Comments
 (0)