Merge pull request opendatahub-io#7158 from jharan1/RHOAIENG-57040-update-status-filter-AAE-models

fix (gen-ai): update Models tab in AAE page to use Ready/Inactive/Unknown status labels

Author: jharan1

packages/gen-ai/frontend/src/app/AIAssets/data/filterOptions.ts

@@ -17,5 +17,5 @@ export const assetsFilterOptions: Record<string, string> = {
 };
 
 export const assetsFilterSelectOptions: Partial<Record<AssetsFilterOptions, string[]>> = {
-  [AssetsFilterOptions.STATUS]: ['Active', 'Inactive'],
+  [AssetsFilterOptions.STATUS]: ['Ready', 'Inactive', 'Unknown'],
 };

packages/gen-ai/frontend/src/app/AIAssets/hooks/__tests__/useAIModelsFilter.spec.ts

@@ -132,7 +132,7 @@ describe('useAIModelsFilter', () => {
   });
 
   describe('Filter by status', () => {
-    it('should filter models by status (select filter)', () => {
+    it('should filter models by Ready status', () => {
       const models = [
         createMockAIModel({ model_id: 'model-1', status: 'Running' }),
         createMockAIModel({ model_id: 'model-2', status: 'Stop' }),
@@ -142,15 +142,15 @@ describe('useAIModelsFilter', () => {
       const { result } = testHook(useAIModelsFilter)(models);
 
       act(() => {
-        result.current.onFilterUpdate(AssetsFilterOptions.STATUS, 'Active');
+        result.current.onFilterUpdate(AssetsFilterOptions.STATUS, 'Ready');
       });
 
       expect(result.current.filteredModels).toHaveLength(2);
       expect(result.current.filteredModels[0].model_id).toBe('model-1');
       expect(result.current.filteredModels[1].model_id).toBe('model-3');
     });
 
-    it('should filter for inactive models', () => {
+    it('should filter models by Inactive status', () => {
       const models = [
         createMockAIModel({ model_id: 'model-1', status: 'Running' }),
         createMockAIModel({ model_id: 'model-2', status: 'Stop' }),
@@ -165,6 +165,23 @@ describe('useAIModelsFilter', () => {
       expect(result.current.filteredModels).toHaveLength(1);
       expect(result.current.filteredModels[0].model_id).toBe('model-2');
     });
+
+    it('should filter models by Unknown status', () => {
+      const models = [
+        createMockAIModel({ model_id: 'model-1', status: 'Running' }),
+        createMockAIModel({ model_id: 'model-2', status: 'Stop' }),
+        createMockAIModel({ model_id: 'model-3', status: 'Pending' }),
+      ];
+
+      const { result } = testHook(useAIModelsFilter)(models);
+
+      act(() => {
+        result.current.onFilterUpdate(AssetsFilterOptions.STATUS, 'Unknown');
+      });
+
+      expect(result.current.filteredModels).toHaveLength(1);
+      expect(result.current.filteredModels[0].model_id).toBe('model-3');
+    });
   });
 
   describe('Multiple filters', () => {

packages/gen-ai/frontend/src/app/AIAssets/hooks/useAIModelsFilter.ts

@@ -4,7 +4,16 @@ import { AssetsFilterOptions, assetsFilterSelectOptions } from '~/app/AIAssets/d
 
 export type FilterData = Record<string, string | string[] | undefined>;
 
-const getStatusLabel = (status: string): string => (status === 'Running' ? 'Active' : 'Inactive');
+const getStatusLabel = (status: string): string => {
+  switch (status) {
+    case 'Running':
+      return 'Ready';
+    case 'Stop':
+      return 'Inactive';
+    default:
+      return 'Unknown';
+  }
+};
 
 const isSelectFilter = (filterType: string): boolean => filterType in assetsFilterSelectOptions;