Labels editing and validation

frontend/src/__tests__/cypress/cypress/pages/modelRegistry/modelVersionDetails.ts

@@ -126,6 +126,38 @@ class ModelVersionDetails {
       this.findTable().find(`[data-label=Key]`).contains(name).parents('tr'),
     );
   }
+
+  findEditLabelsButton() {
+    return cy.findByTestId('editable-labels-group-edit');
+  }
+
+  findAddLabelButton() {
+    return cy.findByTestId('add-label-button');
+  }
+
+  findLabelInput(label: string) {
+    return cy.findByTestId(`edit-label-input-${label}`);
+  }
+
+  findLabel(label: string) {
+    return cy.findByTestId(`editable-label-${label}`);
+  }
+
+  findRemoveLabelButton(label: string) {
+    return cy.findByTestId(`remove-label-${label}`);
+  }
+
+  findLabelErrorAlert() {
+    return cy.findByTestId('label-error-alert');
+  }
+
+  findSaveLabelsButton() {
+    return cy.findByTestId('editable-labels-group-save');
+  }
+
+  findDiscardLabelsButton() {
+    return cy.findByTestId('editable-labels-group-discard');
+  }
 }
 
 class PropertyRow extends TableRow {}

frontend/src/__tests__/cypress/cypress/tests/mocked/modelRegistry/modelVersionDetails.cy.ts

@@ -238,19 +238,6 @@ describe('Model version details', () => {
     it('Model version details tab', () => {
       modelVersionDetails.findVersionId().contains('1');
       modelVersionDetails.findDescription().should('have.text', 'Description of model version');
-      modelVersionDetails.findMoreLabelsButton().contains('6 more');
-      modelVersionDetails.findMoreLabelsButton().click();
-      modelVersionDetails.shouldContainsModalLabels([
-        'Testing label',
-        'Financial',
-        'Financial data',
-        'Fraud detection',
-        'Machine learning',
-        'Next data to be overflow',
-        'Label x',
-        'Label y',
-        'Label z',
-      ]);
       modelVersionDetails.findStorageEndpoint().contains('test-endpoint');
       modelVersionDetails.findStorageRegion().contains('test-region');
       modelVersionDetails.findStorageBucket().contains('test-bucket');
@@ -346,6 +333,43 @@ describe('Model version details', () => {
       modelVersionDetails.findModelVersionDropdownItem('Version 2').click();
       modelVersionDetails.findVersionId().contains('2');
     });
+
+    it('should handle label editing', () => {
+      modelVersionDetails.findEditLabelsButton().click();
+      modelVersionDetails.findAddLabelButton().click();
+
+      // Check label exists and is visible
+      cy.findByTestId('editable-label-New Label').should('exist');
+      cy.findByTestId('editable-label-New Label').should('be.visible');
+
+      // Click the label
+      cy.findByTestId('editable-label-New Label').click();
+
+      // Type the new label
+      cy.get('input[data-testid="edit-label-input-New Label"]').should('exist');
+      cy.get('input[data-testid="edit-label-input-New Label"]').should('be.visible');
+      cy.get('input[data-testid="edit-label-input-New Label"]').clear();
+      cy.get('input[data-testid="edit-label-input-New Label"]').type('UniqueLabel123{enter}');
+
+      // Save changes
+      modelVersionDetails.findSaveLabelsButton().click();
+      cy.wait('@UpdatePropertyRow');
+    });
+
+    it('should handle label validation', () => {
+      modelVersionDetails.findEditLabelsButton().click();
+
+      cy.findByTestId('editable-label-Testing label').click();
+
+      cy.get('input[data-testid="edit-label-input-Testing label"]')
+        .should('exist')
+        .should('be.visible');
+
+      const longLabel = 'a'.repeat(64);
+      modelVersionDetails.findLabelInput('Testing label').clear().type(`${longLabel}{enter}`);
+      modelVersionDetails.findLabelErrorAlert().should('contain', "can't exceed 63 characters");
+      modelVersionDetails.findLabel('New Label').should('not.exist');
+    });
   });
 
   describe('Registered deployments tab', () => {

frontend/src/components/DashboardDescriptionListGroup.tsx

@@ -26,6 +26,7 @@ type EditableProps = {
   editButtonTestId?: string;
   saveButtonTestId?: string;
   cancelButtonTestId?: string;
+  discardButtonTestId?: string;
 };
 
 export type DashboardDescriptionListGroupProps = {
@@ -36,6 +37,7 @@ export type DashboardDescriptionListGroupProps = {
   contentWhenEmpty?: React.ReactNode;
   children: React.ReactNode;
   groupTestId?: string;
+  isSaveDisabled?: boolean;
 } & (({ isEditable: true } & EditableProps) | ({ isEditable?: false } & Partial<EditableProps>));
 
 const DashboardDescriptionListGroup: React.FC<DashboardDescriptionListGroupProps> = (props) => {
@@ -57,6 +59,7 @@ const DashboardDescriptionListGroup: React.FC<DashboardDescriptionListGroupProps
     editButtonTestId,
     saveButtonTestId,
     cancelButtonTestId,
+    isSaveDisabled,
   } = props;
   return (
     <DescriptionListGroup data-testid={groupTestId}>
@@ -74,7 +77,7 @@ const DashboardDescriptionListGroup: React.FC<DashboardDescriptionListGroupProps
                         aria-label={`Save edits to ${title}`}
                         variant="link"
                         onClick={onSaveEditsClick}
-                        isDisabled={isSavingEdits}
+                        isDisabled={isSavingEdits || isSaveDisabled}
                       >
                         <CheckIcon />
                       </Button>