feat: allow validation regexp and prohibitRegexp to be an actual RegE…

…xp []

index.d.ts

@@ -142,7 +142,7 @@ export interface IValidation {
   /** Takes min and/or max parameters and validates the range of a value. */
   range?: { max?: number; min?: number }
   /** Takes a string that reflects a JS regex and flags, validates against a string. See JS reference for the parameters. */
-  regexp?: { pattern: string; flags?: string }
+  regexp?: { pattern: string | RegExp; flags?: string }
   /** Validates that there are no other entries that have the same field value at the time of publication. */
   unique?: true
   /** Validates that a value falls within a certain range of dates. */

src/lib/offline-api/validator/schema/field-validations-schema.ts

@@ -21,27 +21,23 @@ const rangeForDate = () =>
     max: Joi.string().optional().allow(null)
   })
 
+const stringOrRegExp = () => Joi.alternatives().try(Joi.string(), Joi.object().instance(RegExp))
+
+const regexpSchema = () =>
+  Joi.object({
+    pattern: stringOrRegExp(),
+    flags: Joi.string().allow(null).optional()
+  })
+
 const linkContentType = validation('linkContentType', Joi.array().items(Joi.string()))
 const inValidation = validation('in', Joi.array())
 const linkMimetypeGroup = validation('linkMimetypeGroup', Joi.array().items(Joi.string()))
 const size = validation('size', range('number'))
 const rangeValidation = validation('range', range('number'))
 
-const regexp = validation(
-  'regexp',
-  Joi.object({
-    pattern: Joi.string(),
-    flags: Joi.string().allow(null).optional()
-  })
-)
+const regexp = validation('regexp', regexpSchema())
 
-const prohibitRegexp = validation(
-  'prohibitRegexp',
-  Joi.object({
-    pattern: Joi.string(),
-    flags: Joi.string().allow(null).optional()
-  })
-)
+const prohibitRegexp = validation('prohibitRegexp', regexpSchema())
 
 const unique = validation('unique', Joi.boolean())
 const dateRange = validation('dateRange', rangeForDate())

test/unit/lib/offline-api/validation/payload-validation-field-validations.spec.ts

@@ -186,6 +186,20 @@ describe('payload validation', function () {
       expect(errors).to.eql([[]])
     })
 
+    it('allows RegExp for pattern', async function () {
+      const errors = await validateBatches(function up(migration) {
+        const person = migration.createContentType('person').name('Person').description('A Person')
+
+        person
+          .createField('fullName')
+          .name('Full Name')
+          .type('Symbol')
+          .validations([{ regexp: { pattern: /^[A-Za-zs]+$/, flags: null } }])
+      }, [])
+
+      expect(errors).to.eql([[]])
+    })
+
     it('can validate all blocks and inlines for RichText', async function () {
       const errors = await validateBatches(function up(migration) {
         const novel = migration