Merge pull request #15644 from rancher/gha-portpr-18558644291-1

[backport v2.12.4] Ensure Compliance --> Profile list loads when profiles contain `skipTests: []`

Author: richard-cox

shell/models/__tests__/compliance.cattle.io.clusterscanprofile.spec.js

@@ -0,0 +1,30 @@
+import ComplianceProfile from '@shell/models/compliance.cattle.io.clusterscanprofile';
+
+describe('class: ComplianceProfile', () => {
+  describe('getter: numberTestsSkipped', () => {
+    it('should return 0 if skipTests is not present in spec', () => {
+      const complianceProfile = new ComplianceProfile({ spec: {} });
+
+      expect(complianceProfile.numberTestsSkipped).toBe(0);
+    });
+
+    it('should return 0 if skipTests is null', () => {
+      const complianceProfile = new ComplianceProfile({ spec: { skipTests: null } });
+
+      expect(complianceProfile.numberTestsSkipped).toBe(0);
+    });
+
+    it('should return 0 if skipTests is an empty array', () => {
+      const complianceProfile = new ComplianceProfile({ spec: { skipTests: [] } });
+
+      expect(complianceProfile.numberTestsSkipped).toBe(0);
+    });
+
+    it('should return the correct number of skipped tests', () => {
+      const tests = ['test-1', 'test-2', 'test-3'];
+      const complianceProfile = new ComplianceProfile({ spec: { skipTests: tests } });
+
+      expect(complianceProfile.numberTestsSkipped).toBe(tests.length);
+    });
+  });
+});

shell/models/compliance.cattle.io.clusterscanprofile.js

@@ -11,7 +11,7 @@ export default class ComplianceProfile extends SteveModel {
   get numberTestsSkipped() {
     const { skipTests = [] } = this.spec;
 
-    return skipTests.length;
+    return skipTests?.length || 0;
   }
 
   get benchmarkVersionLink() {