Skip to content

Commit 99b5ca7

Browse files
committed
chore: fixed test cases
Signed-off-by: Jatin Kumar <jatink.5251@gmail.com>
1 parent 06330e2 commit 99b5ca7

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1189,7 +1189,7 @@ var _ = Describe("FeatureStore Controller", func() {
11891189
Expect(apimeta.FindStatusCondition(resource.Status.Conditions, feastdevv1.RegistryReadyType)).To(BeNil())
11901190
Expect(apimeta.IsStatusConditionTrue(resource.Status.Conditions, feastdevv1.ReadyType)).To(BeFalse())
11911191
Expect(apimeta.IsStatusConditionTrue(resource.Status.Conditions, feastdevv1.OnlineStoreReadyType)).To(BeTrue())
1192-
Expect(apimeta.IsStatusConditionTrue(resource.Status.Conditions, feastdevv1.OfflineStoreReadyType)).To(BeTrue())
1192+
Expect(apimeta.FindStatusCondition(resource.Status.Conditions, feastdevv1.OfflineStoreReadyType)).To(BeNil())
11931193
Expect(resource.Status.Applied.Services.Registry.Remote.FeastRef.Name).To(Equal(referencedRegistry.Name))
11941194
cond = apimeta.FindStatusCondition(resource.Status.Conditions, feastdevv1.ReadyType)
11951195
Expect(cond).NotTo(BeNil())

infra/feast-operator/internal/controller/services/services_test.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,10 @@ func (feast *FeastServices) refreshFeatureStore(ctx context.Context, key types.N
4242
}
4343

4444
func applySpecToStatus(fs *feastdevv1.FeatureStore) {
45-
if fs.Status.Applied.Services == nil && fs.Spec.Services != nil {
45+
if fs.Spec.Services != nil {
4646
fs.Status.Applied.Services = fs.Spec.Services.DeepCopy()
4747
}
48-
if len(fs.Status.Applied.FeastProject) == 0 {
49-
fs.Status.Applied.FeastProject = fs.Spec.FeastProject
50-
}
48+
fs.Status.Applied.FeastProject = fs.Spec.FeastProject
5149
Expect(k8sClient.Status().Update(context.Background(), fs)).To(Succeed())
5250
}
5351

0 commit comments

Comments
 (0)