chore(Canvas): Remove Route name restriction

fix: KaotoIO#968
lordrip · May 3, 2024 · 1dd7bf2 · 1dd7bf2
1 parent 062330e
commit 1dd7bf2
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 31 deletions.
diff --git a/packages/ui/src/components/InlineEdit/routeIdValidator.test.ts b/packages/ui/src/components/InlineEdit/routeIdValidator.test.ts
@@ -37,29 +37,4 @@ describe('routeIdValidator', () => {
     expect(result.status).toEqual(ValidationStatus.Error);
     expect(result.errMessages).toEqual(['Name must be unique']);
   });
-
-  it('should return an error if the name is not a valid URI', () => {
-    const resource = new CamelRouteResource(camelRouteJson);
-    const visualEntities = resource.getVisualEntities();
-    jest.spyOn(visualEntities[0], 'getId').mockReturnValue('flow-1234');
-
-    const result = RouteIdValidator.validateUniqueName('The amazing Route', visualEntities);
-
-    expect(result.status).toEqual(ValidationStatus.Error);
-    expect(result.errMessages).toEqual(['Name should only contain lowercase letters, numbers, and dashes']);
-  });
-
-  it('should return an error if the name is not unique neither a valid URI', () => {
-    const resource = new CamelRouteResource(camelRouteJson);
-    const visualEntities = resource.getVisualEntities();
-    jest.spyOn(visualEntities[0], 'getId').mockReturnValue('The amazing Route');
-
-    const result = RouteIdValidator.validateUniqueName('The amazing Route', visualEntities);
-
-    expect(result.status).toEqual(ValidationStatus.Error);
-    expect(result.errMessages).toEqual([
-      'Name should only contain lowercase letters, numbers, and dashes',
-      'Name must be unique',
-    ]);
-  });
 });
diff --git a/packages/ui/src/components/InlineEdit/routeIdValidator.ts b/packages/ui/src/components/InlineEdit/routeIdValidator.ts
@@ -20,18 +20,13 @@ export class RouteIdValidator {
     const errMessages = [];
     const flowsIds = visualEntities.map((flow) => flow.getId());
 
-    const isValidURI = RouteIdValidator.isNameValidCheck(flowName);
-    if (!isValidURI) {
-      errMessages.push('Name should only contain lowercase letters, numbers, and dashes');
-    }
-
     const isUnique = !flowsIds.includes(flowName);
     if (!isUnique) {
       errMessages.push('Name must be unique');
     }
 
     return {
-      status: isValidURI && isUnique ? ValidationStatus.Success : ValidationStatus.Error,
+      status: isUnique ? ValidationStatus.Success : ValidationStatus.Error,
       errMessages,
     };
   }