monastery 1.37.3 → 1.38.2

package/changelog.md

@@ -2,6 +2,27 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [1.38.2](https://github.com/boycce/monastery/compare/1.38.1...1.38.2) (2022-07-31)
+
+
+### Bug Fixes
+
+* init load time ([2273f11](https://github.com/boycce/monastery/commit/2273f11a654a9cfe05ee15ed682203a8752237d6))
+
+### [1.38.1](https://github.com/boycce/monastery/compare/1.38.0...1.38.1) (2022-06-20)
+
+
+### Bug Fixes
+
+* changed ACL default ([cc92624](https://github.com/boycce/monastery/commit/cc926243728d0752940359fe4ae4a490f21b5750))
+
+## [1.38.0](https://github.com/boycce/monastery/compare/1.37.3...1.38.0) (2022-06-17)
+
+
+### Features
+
+* added options.metadata ([cb4ca43](https://github.com/boycce/monastery/commit/cb4ca43a395c1f90a5e37b8cc95472053e2b3b36))
+
 ### [1.37.3](https://github.com/boycce/monastery/compare/1.37.2...1.37.3) (2022-06-17)
 
 

package/docs/definition/index.md

@@ -212,7 +212,10 @@ Since unique indexes by default don't allow multiple documents with `null`, you
 
 ### Custom field rules
 
-You are able to define custom field rules to use. (`this` will refer to the data object passed in)
+You are able to define custom field rules to use. 
+
+- `this` will refer to the data object passed in
+- by default, custom rules will ignore `undefined` values
 
 ```js
 {
@@ -223,6 +226,9 @@ You are able to define custom field rules to use. (`this` will refer to the data
     },
     // Full definition
     isGrandMaster: {
+      validateUndefined: false,   // default
+      validateNull: true,         // default
+      validateEmptyString: true,  // default
       message: function(value, ruleArgument, path, model) {
         return 'Only grand masters are permitted'
       },

package/docs/image-plugin.md

@@ -18,7 +18,10 @@ To use the default image plugin shipped with monastery, you need to use the opti
       formats: ['bmp', 'gif', 'jpg', 'jpeg', 'png', 'tiff'], // default (use 'any' to allow all extensions)
       getSignedUrl: false, // default (get a S3 signed url after `model.find()`, can be defined per request)
       path: (uid, basename, ext, file) => `/full/${uid}.${ext}`, // default
-      params: {}, // upload params (https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/S3.html#upload-property)
+      metadata: {},
+      // Any s3 upload param, which takes precedence over the params above
+      // https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/S3.html#upload-property)
+      params: {},
     }
   })
 ```
@@ -65,3 +68,21 @@ user.update({
 Due to known limitations, we are not able to verify the contents of non-binary files, only the filename extension (e.g. .txt, .svg) before uploading to S3
 
 ...to be continued
+
+### Image sizes
+
+I've put together a AWS Lambda function which you can use to generate small/medium/large image sizes automatically for any new files uploaded to your bucket.
+[https://github.com/boycce/s3-lambda-thumbnail-generator](https://github.com/boycce/s3-lambda-thumbnail-generator#install-bucket)
+
+You can override the function's default image sizes via the `metadata` option globally in the manager options, or per file:
+```js
+// Per file
+let user = db.model('user', {
+  fields: {
+    logo:  {
+      type: 'image',
+      metadata: { small: '*x300', medium: '*x800', large: '*x1200' },
+    },
+  }
+}
+```

package/docs/model/find.md

@@ -74,7 +74,7 @@ user.find({ query: {...}, populate: ['myBook'] })
 ```
 
 You can also populate within embedded document fields. Although at this time arrays are not supported,
-you would need to use the [example below](#more-control).
+you would need to use the [example below](#populate-multiple-documents-into-virtual-fields).
 ```js
 user.find({ query: {...}, populate: ['myBooks.book'] })
 ```

package/docs/model/insert.md

@@ -14,7 +14,7 @@ Validate and insert document(s) in a collection and calls model hooks: `beforeIn
 - `data` *(object\|array)* - Data that is validated against the model fields. Key names can be in dot or bracket notation which is handy for HTML FormData.
 - [[`blacklist`](#blacklisting)] *(array\|string\|false)*: augment `definition.insertBL`. `false` will remove all blacklisting
 - [`project`] *(string\|array\|object)*: project these fields, ignores blacklisting
-- [`skipValidation`] (string\|array): skip validation for this field name(s)
+- [`skipValidation`] (string\|array\|boolean): skip validation for these field name(s), or `true` for all fields
 - [`timestamps`] *(boolean)*: whether `createdAt` and `updatedAt` are automatically inserted, defaults to `manager.timestamps`
 - [[`any mongodb option`](http://mongodb.github.io/node-mongodb-native/3.2/api/Collection.html#insert)] *(any)*
 

package/docs/model/update.md

@@ -15,7 +15,7 @@ Update document(s) in a collection and calls model hooks: `beforeUpdate`,  `afte
 - [`data`](#data) *(object)* - data that's validated against the model fields (always wrapped in `{ $set: .. }`)
 - [[`blacklist`](#blacklisting)]*(array\|string\|false)*: augment `definition.updateBL`. `false` will remove all blacklisting
 - [`project`] *(string\|array\|object)*: project these fields, ignores blacklisting
-- [`skipValidation`] (string\|array): skip validation for this field name(s)
+- [`skipValidation`] (string\|array\|boolean): skip validation for these field name(s), or `true` for all fields
 - [`sort`] *(string\|object\|array)*: same as the mongodb option, but  allows for string parsing e.g. 'name', 'name:1'
 - [`timestamps`] *(boolean)*: whether `updatedAt` is automatically updated, defaults to the `manager.timestamps` value
 - [[`any mongodb option`](http://mongodb.github.io/node-mongodb-native/3.2/api/Collection.html#update)] *(any)*

package/docs/model/validate.md

@@ -14,7 +14,7 @@ Validate a model and calls the model hook: `beforeValidate`
 
 [`options`] *(object)*
 
-- [`skipValidation`] (string\|array): skip validation for this field name(s)
+- [`skipValidation`] (string\|array\/boolean): skip validation for thse field name(s), or `true` for all fields
 - [[`blacklist`](#blacklisting)] *(array\|string\|false)*: augment the model's blacklist. `false` will remove all blacklisting
 - [`project`] *(string\|array\|object)*: project these fields, ignores blacklisting
 - [`timestamps`] *(boolean)*: whether `createdAt` and `updatedAt` are inserted, or `updatedAt` is updated, depending on the `update` value. Defaults to the `manager.timestamps` value

package/docs/readme.md

@@ -93,12 +93,19 @@ Coming soon...
 - ~~Whitelisting a parent will remove any previously blacklisted children~~
 - ~~Blacklist/project works the same across find/insert/update/validate~~
 - Automatic embedded document ids/createdAt/updatedAt fields
-- ~~Change ACL default 'public read' to 'private'~~
+- ~~Ability to change ACL default on the manager~~
 - ~~Public db.arrayWithSchema method~~
 - Global after/before hooks
+- before hooks can receive a data array, remove this
 - docs: Make the implicit ID query conversion more apparent
 - Split away from Monk (unless updated)
 
+## Versions
+
+- Monk: `v7.3.4`
+- MongoDB NodeJS driver: `v3.2.3` ([compatibility](https://www.mongodb.com/docs/drivers/node/current/compatibility/#compatibility))
+- MongoDB: [`v4.0.0`](https://www.mongodb.com/docs/v4.2/reference/)
+
 ## Special Thanks
 
 [Jerome Gravel-Niquet](https://github.com/jeromegn)

package/lib/rules.js

@@ -1,7 +1,6 @@
 // Todo: remove stringnums in date/number/integer rules
 let ObjectId = require('mongodb').ObjectId
 let util = require('./util')
-let validator = require('validator')
 
 module.exports = {
 
@@ -175,31 +174,31 @@ module.exports = {
   },
   isAfter: {
     message: (x, arg) => 'Value was before the configured time (' + arg + ')',
-    fn: function(x, arg) { return validator.isAfter(x, arg) }
+    fn: function(x, arg) { return require('validator').isAfter(x, arg) }
   },
   isBefore: {
     message: (x, arg) => 'Value was after the configured time (' + arg + ')',
-    fn: function(x, arg) { return validator.isBefore(x, arg) }
+    fn: function(x, arg) { return require('validator').isBefore(x, arg) }
   },
   isCreditCard: {
     message: 'Value was not a valid credit card.',
-    fn: function(x, arg) { return validator.isCreditCard(x, arg) }
+    fn: function(x, arg) { return require('validator').isCreditCard(x, arg) }
   },
   isEmail: {
     message: 'Please enter a valid email address.',
-    fn: function(x, arg) { return validator.isEmail(x, arg) }
+    fn: function(x, arg) { return require('validator').isEmail(x, arg) }
   },
   isHexColor: {
     message: 'Value was not a valid hex color.',
-    fn: function(x, arg) { return validator.isHexColor(x, arg) }
+    fn: function(x, arg) { return require('validator').isHexColor(x, arg) }
   },
   isIn: {
     message: (x, arg) => 'Value was not in the configured whitelist (' + arg.join(', ') + ')',
-    fn: function(x, arg) { return validator.isIn(x, arg) }
+    fn: function(x, arg) { return require('validator').isIn(x, arg) }
   },
   isIP: {
     message: 'Value was not a valid IP address.',
-    fn: function(x, arg) { return validator.isIP(x, arg) }
+    fn: function(x, arg) { return require('validator').isIP(x, arg) }
   },
   isNotEmptyString: {
     validateEmptyString: true,
@@ -210,15 +209,15 @@ module.exports = {
   },
   isNotIn: {
     message: (x, arg) => 'Value was in the configured blacklist (' + arg.join(', ') + ')',
-    fn: function(x, arg) { return !validator.isIn(x, arg) }
+    fn: function(x, arg) { return !require('validator').isIn(x, arg) }
   },
   isURL: {
     message: 'Value was not a valid URL.',
-    fn: function(x, arg) { return validator.isURL(x, arg === true? undefined : arg) }
+    fn: function(x, arg) { return require('validator').isURL(x, arg === true? undefined : arg) }
   },
   isUUID: {
     message: 'Value was not a valid UUID.',
-    fn: function(x, arg) { return validator.isUUID(x) }
+    fn: function(x, arg) { return require('validator').isUUID(x) }
   },
   minLength: {
     message: function(x, arg) {
@@ -227,7 +226,7 @@ module.exports = {
     },
     fn: function(x, arg) {
       if (typeof x !== 'string' && !util.isArray(x)) throw new Error ('Value was not a string or an array.')
-      else if (typeof x === 'string') return validator.isLength(x, arg)
+      else if (typeof x === 'string') return require('validator').isLength(x, arg)
       else return x.length >= arg
     }
   },
@@ -238,14 +237,14 @@ module.exports = {
     },
     fn: function(x, arg) {
       if (typeof x !== 'string' && !util.isArray(x)) throw new Error ('Value was not a string or an array.')
-      else if (typeof x === 'string') return validator.isLength(x, 0, arg)
+      else if (typeof x === 'string') return require('validator').isLength(x, 0, arg)
       else return x.length <= arg
     }
   },
   regex: {
     message: (x, arg) => 'Value did not match the configured regular expression (' + arg + ')',
     fn: function(x, arg) {
-      if (util.isRegex(arg)) return validator.matches(x, arg)
+      if (util.isRegex(arg)) return require('validator').matches(x, arg)
       throw new Error('This rule expects a regular expression to be configured, but instead got the '
         + typeof arg + ' `' + arg + '`.')
     }

package/package.json

@@ -2,7 +2,7 @@
   "name": "monastery",
   "description": "⛪ A straight forward MongoDB ODM built around Monk",
   "author": "Ricky Boyce",
-  "version": "1.37.3",
+  "version": "1.38.2",
   "license": "MIT",
   "repository": "github:boycce/monastery",
   "homepage": "https://boycce.github.io/monastery/",

package/plugins/images/index.js

@@ -1,6 +1,4 @@
-let fileType = require('file-type')
-let nanoid = require('nanoid')
-let S3 = require('aws-sdk/clients/s3')
+// requiring: nanoid, file-type, aws-sdk/clients/s3
 let util = require('../../lib/util')
 
 let plugin = module.exports = {
@@ -9,7 +7,6 @@ let plugin = module.exports = {
     /**
      * Monastery plugin that allows models to automatically process and store uploaded images
      * e.g. fields.avatar: { image: true }
-     * e.g. fields.photos: [{ image: true, //sizes: { large: [800,  600], .. } (not implemented) }]
      *
      * Note we cannot accurately test for non binary file-types, e.g. 'txt', 'csv'
      *
@@ -25,7 +22,7 @@ let plugin = module.exports = {
     }
 
     // Settings
-    this.awsAcl = options.awsAcl || 'private' // default
+    this.awsAcl = options.awsAcl || 'public-read' // default
     this.awsBucket = options.awsBucket
     this.awsAccessKeyId = options.awsAccessKeyId
     this.awsSecretAccessKey = options.awsSecretAccessKey
@@ -34,6 +31,7 @@ let plugin = module.exports = {
     this.formats = options.formats || ['bmp', 'gif', 'jpg', 'jpeg', 'png', 'tiff']
     this.getSignedUrl = options.getSignedUrl
     this.manager = manager
+    this.metadata = options.metadata ? util.deepCopy(options.metadata) : undefined,
     this.params = options.params ? util.deepCopy(options.params) : {},
     this.path = options.path || function (uid, basename, ext, file) { return `full/${uid}.${ext}` }
 
@@ -43,15 +41,17 @@ let plugin = module.exports = {
       return
     }
 
-    // Create s3 'service' instance
+    // Create s3 'service' instance (defer require since it takes 120ms to load)
     // https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/S3.html#constructor-property
     manager._getSignedUrl = this._getSignedUrl
-    this.s3 = new S3({
-      credentials: {
-        accessKeyId: this.awsAccessKeyId,
-        secretAccessKey: this.awsSecretAccessKey
-      }
-    })
+    this.s3 = () => {
+      return this._s3 || (this._s3 = new (require('aws-sdk/clients/s3'))({
+        credentials: {
+          accessKeyId: this.awsAccessKeyId,
+          secretAccessKey: this.awsSecretAccessKey
+        }
+      }))
+    }
 
     // Add before model hook
     manager.beforeModel.push(this.setupModel.bind(this))
@@ -140,15 +140,15 @@ let plugin = module.exports = {
       return Promise.all(files.map(filesArr => {
         return Promise.all(filesArr.map(file => {
           return new Promise((resolve, reject) => {
-            let uid = nanoid.nanoid()
+            let uid = require('nanoid').nanoid()
             let path = filesArr.imageField.path || plugin.path
             let image = {
               bucket: filesArr.imageField.awsBucket || plugin.awsBucket,
               date: plugin.manager.useMilliseconds? Date.now() : Math.floor(Date.now() / 1000),
               filename: file.name,
               filesize: file.size,
+              metadata: filesArr.imageField.metadata || plugin.metadata,
               path: path(uid, file.name, file.ext, file),
-              // sizes: ['large', 'medium', 'small'],
               uid: uid,
             }
             let s3Options = {
@@ -158,6 +158,7 @@ let plugin = module.exports = {
               Body: file.data,
               Bucket: image.bucket,
               Key: image.path,
+              Metadata: image.metadata,
               ...(filesArr.imageField.params || plugin.params),
             }
             plugin.manager.info(
@@ -167,7 +168,7 @@ let plugin = module.exports = {
               plugin._addImageObjectsToData(filesArr.inputPath, data, image)
               resolve(s3Options)
             } else {
-              plugin.s3.upload(s3Options, (err, response) => {
+              plugin.s3().upload(s3Options, (err, response) => {
                 if (err) return reject(err)
                 plugin._addImageObjectsToData(filesArr.inputPath, data, image)
                 resolve(s3Options)
@@ -347,11 +348,12 @@ let plugin = module.exports = {
       if (useCount[key] > 0) continue
       let pre = preExistingImages.find(o => o.image.uid == key)
       unused.push(
-        // original key can have a different extension
+        // original key can have a different extension, but always expect generated assets
+        // to be in jpg
         { Key: pre.image.path },
         { Key: `small/${key}.jpg` },
         { Key: `medium/${key}.jpg` },
-        { Key: `large/${key}.jpg` }
+        { Key: `large/${key}.jpg` },
       )
       plugin.manager.info(
         `Removing '${pre.image.filename}' from '${pre.image.bucket}/${pre.image.path}'`
@@ -362,7 +364,7 @@ let plugin = module.exports = {
     // the file doesnt get deleted, we only delete from plugin.awsBucket.
     if (!unused.length) return
     await new Promise((resolve, reject) => {
-      plugin.s3.deleteObjects({
+      plugin.s3().deleteObjects({
         Bucket: plugin.awsBucket,
         Delete: { Objects: unused }
       }, (err, data) => {
@@ -428,7 +430,7 @@ let plugin = module.exports = {
     return Promise.all(validFiles.map(filesArr => {
       return Promise.all(filesArr.map((file, i) => {
         return new Promise((resolve, reject) => {
-          fileType.fromBuffer(file.data).then(res => {
+          require('file-type').fromBuffer(file.data).then(res => {
             let filesize = filesArr.imageField.filesize || plugin.filesize
             let formats = filesArr.imageField.formats || plugin.formats
             let allowAny = util.inArray(formats, 'any')
@@ -504,6 +506,7 @@ let plugin = module.exports = {
           fullPath: path2,
           fullPathRegex: new RegExp('^' + path2.replace(/\.[0-9]+/g, '.[0-9]+').replace(/\./g, '\\.') + '$'),
           getSignedUrl: field.getSignedUrl,
+          metadata: field.metadata ? util.deepCopy(field.metadata) : undefined,
           path: field.path,
           params: field.params ? util.deepCopy(field.params) : undefined,
         })
@@ -513,6 +516,7 @@ let plugin = module.exports = {
           date: { type: 'number' },
           filename: { type: 'string' },
           filesize: { type: 'number' },
+          metadata: { type: 'any' },
           path: { type: 'string' },
           schema: { image: true, nullObject: true, isImageObject: true },
           uid: { type: 'string' },
@@ -573,7 +577,7 @@ let plugin = module.exports = {
      * @param {number} <bucket>
      * @see https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/S3.html#getSignedUrl-property
      */
-    let signedUrl = plugin.s3.getSignedUrl('getObject', {
+    let signedUrl = plugin.s3().getSignedUrl('getObject', {
       Bucket: bucket || plugin.awsBucket,
       Expires: expires,
       Key: path,

package/test/crud.js

@@ -213,6 +213,45 @@ module.exports = function(monastery, opendb) {
     db.close()
   })
 
+  test('update general max', async () => {
+    let db = (await opendb(null)).db
+    let user = db.model('user', {
+      fields: {
+        name: { type: 'string' },
+        active: { type: 'boolean' },
+      },
+    })
+
+    // Insert
+    let inserted = await user.insert({ data: { name: 'Martin Luther' }})
+
+    expect(inserted).toEqual({
+      _id: expect.any(Object),
+      name: 'Martin Luther',
+    })
+
+    let updatedUser1 = await user.update({
+      query: { _id: inserted._id },
+      data: { name: 'Martin' },
+      blacklist: ['active'],
+    })
+
+    expect(updatedUser1).toEqual({ name: 'Martin' })
+
+    let updatedUser2 = await user._update(
+      { _id: db.id(inserted._id) },
+      { $set: { name: 'Martin2' }},
+    )
+
+
+    expect(updatedUser2).toEqual({ n: 1, nModified: 1, ok: 1 })
+
+    // Mongo connected?
+    // Try native updates
+
+    db.close()
+  })
+
   test('update defaults', async () => {
     let db = (await opendb(null, { useMilliseconds: true, serverSelectionTimeoutMS: 2000 })).db
     let user = db.model('user', {

package/test/plugin-images.js

@@ -75,6 +75,7 @@ module.exports = function(monastery, opendb) {
       date: { type: 'number', isNumber: true },
       filename: { type: 'string', isString: true },
       filesize: { type: 'number', isNumber: true },
+      metadata: { type: 'any', isAny: true },
       path: { type: 'string', isString: true },
       schema: {
         type: 'object', isObject: true, image: true, nullObject: true, default: undefined,
@@ -641,10 +642,11 @@ module.exports = function(monastery, opendb) {
     app.use(upload({ limits: { fileSize: 1000 * 480, files: 10 }}))
 
     // Basic tests
-    expect(plugin.awsAcl).toEqual('private')
+    expect(plugin.awsAcl).toEqual('public-read')
     expect(plugin.filesize).toEqual(undefined)
     expect(plugin.formats).toEqual(['bmp', 'gif', 'jpg', 'jpeg', 'png', 'tiff'])
     expect(plugin.getSignedUrl).toEqual(undefined)
+    expect(plugin.metadata).toEqual(undefined)
     expect(plugin.path).toEqual(expect.any(Function))
     expect(plugin.params).toEqual({})
 
@@ -688,7 +690,7 @@ module.exports = function(monastery, opendb) {
         // S3 options
         expect(response[1]).toEqual([
           [{
-            ACL: 'private',
+            ACL: 'public-read',
             Body: expect.any(Object),
             Bucket: 'fake',
             Key: expect.stringMatching(/^full\/.*png$/),
@@ -853,7 +855,7 @@ module.exports = function(monastery, opendb) {
     db.close()
   })
 
-  test('images options awsAcl, awsBucket, params, path', async () => {
+  test('images options awsAcl, awsBucket, metadata, params, path', async () => {
     let db = (await opendb(null, {
       timestamps: false,
       serverSelectionTimeoutMS: 2000,
@@ -862,6 +864,7 @@ module.exports = function(monastery, opendb) {
         awsBucket: 'fake',
         awsAccessKeyId: 'fake',
         awsSecretAccessKey: 'fake',
+        metadata: { small: '*x300' , medium: '*x800', large: '*x1200' },
         params: { ContentLanguage: 'DE'},
         path: (uid, basename, ext, file) => `images/${basename}`,
       }
@@ -874,6 +877,7 @@ module.exports = function(monastery, opendb) {
           type: 'image',
           awsAcl: 'public-read-write',
           awsBucket: 'fake2',
+          metadata: { small: '*x100' , medium: '*x400', large: '*x800' },
           params: { ContentLanguage: 'NZ'},
           path: (uid, basename, ext, file) => `images2/${basename}`,
         },
@@ -904,6 +908,7 @@ module.exports = function(monastery, opendb) {
             date: expect.any(Number),
             filename: 'logo.png',
             filesize: expect.any(Number),
+            metadata: { small: '*x300' , medium: '*x800', large: '*x1200' },
             path: 'images/logo.png',
             uid: expect.any(String),
           },
@@ -912,6 +917,7 @@ module.exports = function(monastery, opendb) {
             date: expect.any(Number),
             filename: 'logo2.png',
             filesize: expect.any(Number),
+            metadata: { small: '*x100' , medium: '*x400', large: '*x800' },
             path: 'images2/logo2.png',
             uid: expect.any(String),
           },
@@ -924,6 +930,7 @@ module.exports = function(monastery, opendb) {
             Bucket: 'fake',
             ContentLanguage: 'DE',
             Key: 'images/logo.png',
+            Metadata:  { small: '*x300' , medium: '*x800', large: '*x1200' },
           }],
           [{
             ACL: 'public-read-write',
@@ -931,6 +938,7 @@ module.exports = function(monastery, opendb) {
             Bucket: 'fake2',
             ContentLanguage: 'NZ',
             Key: 'images2/logo2.png',
+            Metadata: { small: '*x100' , medium: '*x400', large: '*x800' },
           }],
         ])
         res.send()

package/test/validate.js

@@ -816,12 +816,14 @@ module.exports = function(monastery, opendb) {
     let user = db.model('user', { fields: {
       name: { type: 'string', minLength: 7 },
       email: { type: 'string', isEmail: true },
-      names: { type: 'string', enum: ['Martin', 'Luther'] },
       amount: { type: 'number' },
     }})
     let user2 = db.model('user', { fields: {
       amount: { type: 'number', required: true },
     }})
+    let user3 = db.model('user', { fields: {
+      names: { type: 'string', enum: ['Martin', 'Luther'], default: 'Martin' },
+    }})
 
     // MinLength
     await expect(user.validate({ name: 'Martin Luther' })).resolves.toEqual({name: 'Martin Luther'})
@@ -850,8 +852,9 @@ module.exports = function(monastery, opendb) {
     })
 
     // Enum
-    await expect(user.validate({ names: 'Martin' })).resolves.toEqual({ names: 'Martin' })
-    await expect(user.validate({ names: 'bad name' })).rejects.toContainEqual({
+    await expect(user3.validate({})).resolves.toEqual({ names: 'Martin' })
+    await expect(user3.validate({ names: 'Luther' })).resolves.toEqual({ names: 'Luther' })
+    await expect(user3.validate({ names: 'bad name' })).rejects.toContainEqual({
       detail: 'Invalid enum value',
       status: '400',
       title: 'names',