Added edge case for re-enabling source on the MC settings page (#2132)

* added edge case for re-enabling source on the MC settings page

Signed-off-by: Yulia Krimerman <juliapiterova@hotmail.com>

* added another test case for this scenario

Signed-off-by: Yulia Krimerman <juliapiterova@hotmail.com>

---------

Signed-off-by: Yulia Krimerman <juliapiterova@hotmail.com>

Author: YuliaKrimerman

clients/ui/frontend/src/__tests__/cypress/cypress/tests/mocked/modelCatalogSettings/modelCatalogSettings.cy.ts

@@ -469,6 +469,20 @@ describe('Catalog Source Configs Table', () => {
       row.shouldHaveValidationStatus('Starting');
     });
 
+    it('should show "Starting" status when re-enabling source (enabled=true but status=disabled)', () => {
+      const reenablingSource = mockCatalogSource({
+        id: 'hf-google',
+        name: 'HuggingFace Google',
+        status: 'disabled',
+      });
+      setupMocks([reenablingSource], { catalogs: [huggingFaceSource] }); // huggingFaceSource has enabled=true
+      modelCatalogSettings.visit();
+      const row = modelCatalogSettings.getRow('HuggingFace Google');
+      row.findName().should('be.visible');
+      row.shouldHaveValidationStatus('Starting');
+      row.findValidationStatus().findByTestId('source-status-starting-hf-google').should('exist');
+    });
+
     it('should show "Failed" status with error message for error sources', () => {
       const errorSource = mockCatalogSource({
         id: 'hf-google',

clients/ui/frontend/src/app/pages/modelCatalogSettings/components/CatalogSourceStatus.tsx

@@ -99,8 +99,9 @@ const CatalogSourceStatus: React.FC<CatalogSourceStatusProps> = ({ catalogSource
     }
 
     case CatalogSourceStatusEnum.DISABLED:
-      return <>-</>;
-
+      // If we reach here, config.enabled is true - line 24
+      // But status is still DISABLED, so show "Starting" (re-enable case)
+      return startingOrUnknownLabel;
     default:
       return startingOrUnknownLabel;
   }