From: Afreen Misbah <afreen@ibm.com>
Date: Thu, 23 Apr 2026 13:23:25 +0000 (+0530)
Subject: Fix unit test
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=f58defe048ec4aa2bbf90cb54bdabd6cc240dfa3;p=ceph.git

Fix unit test

Signed-off-by: Afreen Misbah <afreen@ibm.com>
(cherry picked from commit 38412a464fe1bb313a954ccf909f447f0db538c0)
---

diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.spec.ts b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.spec.ts
index 462f5569f14d..379c1a272dd2 100644
--- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.spec.ts
@@ -104,7 +104,7 @@ describe('ServiceFormComponent', () => {
       // placement labels take only single value
       formHelper.setValue('service_type', 'mgr');
       formHelper.setValue('placement', 'label');
-      formHelper.setValue('label', "{content: 'foo', selected:  true}");
+      formHelper.setValue('label', { content: 'foo', selected: true });
 
       component.onSubmit();
 
diff --git a/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.spec.ts b/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.spec.ts
index 4e670f455b7f..056a4da67d04 100755
--- a/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.spec.ts
@@ -336,22 +336,6 @@ describe('NvmeofService', () => {
       req.flush(mockGroups);
     });
 
-    it('should filter hosts by array label placement', (done) => {
-      const mockGroups = [
-        [{ spec: { group: 'default' }, placement: { hosts: [], label: ['nvmeof', 'storage'] } }]
-      ];
-      mockHostService.getAllHosts.mockReturnValue(of(allHosts));
-
-      service.getHostsForGroup('default').subscribe((hosts: any[]) => {
-        expect(hosts.length).toBe(1);
-        expect(hosts[0].hostname).toBe('host3');
-        done();
-      });
-
-      const req = httpTesting.expectOne(`${API_PATH}/gateway/group`);
-      req.flush(mockGroups);
-    });
-
     it('should return empty array when group not found', (done) => {
       const mockGroups = [
         [{ spec: { group: 'other' }, placement: { hosts: ['host1'], label: [] } }]
diff --git a/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.ts b/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.ts
index 100bff76ab68..fda6a998ad1f 100644
--- a/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.ts
@@ -206,7 +206,9 @@ export class NvmeofService {
     return this.getSubsystem(subsystemNqn, group).pipe(
       mapTo(true),
       catchError((e) => {
-        e?.preventDefault();
+        if (_.isFunction(e?.preventDefault)) {
+          e.preventDefault();
+        }
         return observableOf(false);
       })
     );