fix(rbac): conditional access form validation (janus-idp#1699)

Signed-off-by: Yi Cai <[email protected]>

Author: ciiay

plugins/rbac/src/__fixtures__/mockConditionRules.ts

@@ -91,7 +91,8 @@ export const mockConditionRules = [
           properties: {
             kinds: {
               type: 'array',
-              items: { type: 'string' },
+              items: { type: 'string' }, // TODO: should be enum?
+              minItems: 1,
               description: 'List of kinds to match at least one of',
             },
           },
@@ -110,6 +111,7 @@ export const mockConditionRules = [
             claims: {
               type: 'array',
               items: { type: 'string' },
+              minItems: 1,
               description:
                 'List of claims to match at least one on within ownedBy',
             },

plugins/rbac/src/components/ConditionalAccess/ConditionsFormRowFields.tsx

@@ -123,14 +123,14 @@ export const ConditionsFormRowFields = ({
         {schema ? (
           <Form
             schema={paramsSchema}
-            formData={oldCondition?.params ?? null}
+            formData={oldCondition?.params || {}}
             validator={validator}
             uiSchema={uiSchema}
             fields={customFields}
             onChange={data =>
               handleConditionChange({
                 ...oldCondition,
-                params: data.formData ?? {},
+                params: data.formData || {},
               })
             }
             transformErrors={errors => {

plugins/rbac/src/components/ConditionalAccess/CustomArrayField.tsx

@@ -35,8 +35,9 @@ export const CustomArrayField = (props: FieldProps) => {
         label={name}
         value={fieldVal}
         onChange={e => {
-          setFieldVal(e.target.value);
-          onChange(e.target.value.split(',').map(val => val.trim()));
+          const value = e.target.value;
+          setFieldVal(value);
+          onChange(value ? value.split(',').map(val => val.trim()) : []);
         }}
         className={classes.bgPaper}
         required={required}