fix: trim role names before validation (#8277)

This trims role names before validation and subsequent validation.
This fixes a bug where you could have names that were empty or that
were duplicates of other names, but with leading or trailing
spaces. (They display the same in the UI).

This does not modify how we handle descriptions in the API. While the
UI form requires you to enter a description, the API does not. As
such, we can't make that required now without it being a breaking
change.

src/lib/schema/role-schema.ts

@@ -13,7 +13,7 @@ export const permissionRoleSchema = joi
 export const roleSchema = joi
     .object()
     .keys({
-        name: joi.string().required(),
+        name: joi.string().trim().required(),
         description: joi.string().optional().allow('').allow(null).default(''),
         permissions: joi
             .array()

src/lib/services/access-service.test.ts

@@ -97,6 +97,33 @@ test('should accept empty permissions', async () => {
     });
 });
 
+test('should not accept empty names', async () => {
+    const { accessService } = getSetup();
+    const withWhitespaceName: IRoleValidation = {
+        name: '    ',
+        description: 'description',
+        permissions: [],
+    };
+
+    await expect(
+        accessService.validateRole(withWhitespaceName),
+    ).rejects.toThrow('"name" is not allowed to be empty');
+});
+
+test('should trim leading and trailing whitespace from names', async () => {
+    const { accessService } = getSetup();
+    const withUntrimmedName: IRoleValidation = {
+        name: '   untrimmed ',
+        description: 'description',
+        permissions: [],
+    };
+    expect(await accessService.validateRole(withUntrimmedName)).toEqual({
+        name: 'untrimmed',
+        description: 'description',
+        permissions: [],
+    });
+});
+
 test('should complete environment field of permissions when not present', async () => {
     const { accessService } = getSetup();
     const withoutEnvironmentInPermissions: IRoleValidation = {