fixup! try to fix the test on SQL dbs

Signed-off-by: Miroslav Bajtoš <[email protected]>

test/manipulation.test.js

@@ -2664,18 +2664,31 @@ describe('manipulation', function() {
     });
 
     it('preserves custom type of auto-generated id property', async () => {
-      const userId = 'custom user id';
+      // NOTE: This test is trying to reproduce the behavior observed
+      // when using property defined as follows:
+      //  {type: 'string', generated: true, mongodb: {dataType: 'ObjectID'}}
+      // We want to test that behavior for all connectors, which is tricky,
+      // because not all connectors support autogenerated string PK values.
 
-      const User = db.define('User', {
+      const User = db.define('UserWithStringId', {
         id: {
           type: String,
           id: true,
-          generated: true,
           useDefaultIdType: false,
+          // `useDefaultIdType` is applied only when `generated: true`
+          generated: true,
         },
         name: String,
       }, {forceId: false});
 
+      // disable `generated: true` because many SQL databases cannot
+      // auto-generate string ids
+      User.definition.properties.id.generated = false;
+      User.definition.rawProperties.id.generated = false;
+      await db.automigrate(User.modelName);
+
+      const userId = 'custom user id';
+
       const createdUser = await User.create({id: userId, name: 'testUser'});
       // strict equality check
       createdUser.id.should.equal(userId);