chore: update dependency sinon to v20

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

.github/workflows/ci.yml

@@ -8,7 +8,7 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     env:
       MYSQL_HOST: '127.0.0.1'
       MYSQL_USER: myUser

CHANGES.md

@@ -1,3 +1,119 @@
+2025-03-18, Version 7.1.1
+=========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.1.6 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.2.2 (renovate[bot])
+
+ * chore: update dependency @commitlint/config-conventional to ^19.8.0 (renovate[bot])
+
+
+2025-02-12, Version 7.1.0
+=========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.1.5 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.2.1 (renovate[bot])
+
+ * chore: update dependency @commitlint/config-conventional to ^19.7.1 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.2.0 (renovate[bot])
+
+ * feat: query to fetch unique columns (Muhammad Aaqil)
+
+ * chore: update dependency mocha to ^11.1.0 (renovate[bot])
+
+ * chore: update dependency debug to ^4.4.0 (renovate[bot])
+
+
+2024-12-05, Version 7.0.17
+==========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.1.3 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.1.12 (renovate[bot])
+
+ * chore: update dependency mocha to v11 (renovate[bot])
+
+ * chore: update dependency @commitlint/config-conventional to ^19.6.0 (renovate[bot])
+
+
+2024-11-12, Version 7.0.16
+==========================
+
+ * chore: update dependency loopback-connector to ^6.1.11 (renovate[bot])
+
+ * chore: update dependency loopback-datasource-juggler to ^5.1.2 (renovate[bot])
+
+ * chore: update dependency mocha to ^10.8.2 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.1.10 (renovate[bot])
+
+
+2024-10-11, Version 7.0.15
+==========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.1.1 (renovate[bot])
+
+ * chore: update dependency sinon to v19 (renovate[bot])
+
+ * chore: update dependency ubuntu to v24 (renovate[bot])
+
+ * chore: update dependency eslint to ^8.57.1 (renovate[bot])
+
+ * chore: update dependency sinon to ^18.0.1 (renovate[bot])
+
+ * chore: update dependency @commitlint/config-conventional to ^19.5.0 (renovate[bot])
+
+
+2024-09-09, Version 7.0.14
+==========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.1.0 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.1.9 (renovate[bot])
+
+ * chore: update dependency debug to ^4.3.7 (renovate[bot])
+
+ * fix: enable migration for enum property (Muhammad Aaqil)
+
+ * fix: change condition to treat tinyint as boolean (Muhammad Aaqil)
+
+ * chore: update dependency async to ^3.2.6 (renovate[bot])
+
+ * chore: update dependency @commitlint/config-conventional to ^19.4.1 (renovate[bot])
+
+ * fix: mark primary key auto-increment only if column type is TINYINT SMALLINT MEDIUMINT INT BIGINT (Muhammad Aaqil)
+
+ * chore: update dependency loopback-connector to ^6.1.8 (renovate[bot])
+
+
+2024-08-12, Version 7.0.13
+==========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.0.12 (renovate[bot])
+
+ * chore: update dependency mocha to ^10.7.3 (renovate[bot])
+
+ * chore: update dependency debug to ^4.3.6 (renovate[bot])
+
+ * chore: update dependency mocha to ^10.7.0 (renovate[bot])
+
+
+2024-07-04, Version 7.0.12
+==========================
+
+ * chore: update dependency loopback-datasource-juggler to ^5.0.11 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.1.7 (renovate[bot])
+
+ * chore: update dependency mocha to ^10.6.0 (renovate[bot])
+
+ * chore: update dependency mocha to ^10.5.2 (renovate[bot])
+
+ * chore: update dependency loopback-connector to ^6.1.6 (renovate[bot])
+
+
 2024-06-06, Version 7.0.11
 ==========================
 

lib/discovery.js

@@ -301,6 +301,29 @@ function mixinDiscovery(MySQL, mysql) {
     return sql;
   };
 
+  /**
+   * Discover unique keys for a given table
+   * @param {String} table The table name
+   * @param {Object} options The options for discovery
+   */
+
+  /*!
+   * Retrieves a list of column names that have unique key index
+   * @param schema
+   * @param table
+   * @returns {string}
+   */
+  MySQL.prototype.buildQueryUniqueKeys = function(schema, table) {
+    const sql = 'SELECT Column_name AS "columnName",' +
+    ' table_schema AS "owner",' +
+    ' table_name AS "tableName"' +
+    ' FROM Information_schema.statistics' +
+    ' WHERE Table_schema = ' + mysql.escape(schema) +
+    ' AND Table_name = ' + mysql.escape(table) +
+    ' AND Non_unique = 0 AND Index_name <> \'PRIMARY\';';
+    return sql;
+  };
+
   /**
    * Discover foreign keys that reference to the primary key of this table
    * @param {String} table The table name
@@ -358,8 +381,8 @@ function mixinDiscovery(MySQL, mysql) {
         }
         return 'Binary';
       case 'TINYINT':
-        // treat TINYINT(1) as boolean as it is aliased as BOOL and BOOLEAN in mysql
+        // treat TINYINT as boolean as it is aliased as BOOL and BOOLEAN in mysql
-        if (!options.treatTINYINT1AsTinyInt && columnType === 'tinyint(1)') {
+        if (!options.treatTINYINT1AsTinyInt && columnType.includes('tinyint')) {
           return 'Boolean';
         }
       case 'SMALLINT':

lib/migration.js

@@ -628,7 +628,13 @@ function mixinMigration(MySQL, mysql) {
       if (pks.length === 1) {
         const idName = this.idName(model);
         const idProp = this.getModelDefinition(model).properties[idName];
-        if (idProp.generated) {
+        const idColumnType = this.columnDataType(model, idName);
+        if (idProp.generated && (
+          idColumnType === 'TINYINT' ||
+          idColumnType === 'SMALLINT' ||
+          idColumnType === 'MEDIUMINT' ||
+          idColumnType === 'INT' ||
+          idColumnType === 'BIGINT')) {
           sql.push(self.columnEscaped(model, idName) + ' ' +
             self.buildColumnDefinition(model, idName) + ' AUTO_INCREMENT PRIMARY KEY');
         } else {
@@ -764,10 +770,21 @@ function mixinMigration(MySQL, mysql) {
     const colLength = columnMetadata && columnMetadata.dataLength || prop.length || prop.limit;
     const colPrecision = columnMetadata && columnMetadata.dataPrecision;
     const colScale = columnMetadata && columnMetadata.dataScale;
+    let enumList = '';
+    if (colType && colType === 'ENUM') {
+      if (prop.jsonSchema && prop.jsonSchema.enum) {
+        prop.jsonSchema.enum.forEach(item => {
+          enumList += `'${item}',`;
+        });
+        // remove trailing comma
+        enumList = enumList.substring(0, enumList.length - 1);
+      }
+    }
     // info on setting column specific properties
     // i.e dataLength, dataPrecision, dataScale
     // https://loopback.io/doc/en/lb3/Model-definition-JSON-file.html
     if (colType) {
+      if (colType === 'ENUM') return colType + '(' + enumList + ')';
       if (colLength) return colType + '(' + colLength + ')';
       if (colPrecision && colScale) return colType + '(' + colPrecision + ',' + colScale + ')';
       if (colPrecision) return colType + '(' + colPrecision + ')';

package.json

@@ -1,6 +1,6 @@
 {
   "name": "loopback-connector-mysql",
-  "version": "7.0.11",
+  "version": "7.1.1",
   "description": "MySQL connector for loopback-datasource-juggler",
   "engines": {
     "node": ">=18"
@@ -21,23 +21,23 @@
     "setup.sh"
   ],
   "dependencies": {
-    "async": "^3.2.5",
+    "async": "^3.2.6",
-    "debug": "^4.3.5",
+    "debug": "^4.4.0",
     "lodash": "^4.17.21",
-    "loopback-connector": "^6.1.5",
+    "loopback-connector": "^6.2.2",
     "mysql2": "^3.6.3",
     "patch-package": "^8.0.0",
     "strong-globalize": "^6.0.6"
   },
   "devDependencies": {
-    "@commitlint/config-conventional": "^19.2.2",
+    "@commitlint/config-conventional": "^19.8.0",
-    "eslint": "^8.57.0",
+    "eslint": "^8.57.1",
     "eslint-config-loopback": "^13.1.0",
-    "loopback-datasource-juggler": "^5.0.10",
+    "loopback-datasource-juggler": "^5.1.6",
-    "mocha": "^10.4.0",
+    "mocha": "^11.1.0",
     "rc": "^1.2.8",
     "should": "^13.2.3",
-    "sinon": "^18.0.0"
+    "sinon": "^20.0.0"
   },
   "repository": {
     "type": "git",

test/migration.test.js

@@ -85,6 +85,13 @@ describe('migrations', function() {
           Key: '',
           Default: null,
           Extra: ''},
+        typings: {
+          Field: 'typings',
+          Type: "enum('A','B')",
+          Null: 'YES',
+          Key: '',
+          Default: null,
+          Extra: ''},
       });
       done();
     });
@@ -548,9 +555,10 @@ function setup(done) {
   require('./init.js');
 
   db = global.getSchema();
-
+  const customType = db.EnumFactory('A', 'B');
   UserData = db.define('UserData', {
     email: {type: String, null: false, index: true},
+    typings: {type: customType},
     name: String,
     bio: Schema.Text,
     birthDate: Date,

test/mysql.discover.test.js

@@ -283,6 +283,21 @@ describe('Discover LDL schema from a table', function() {
   });
 });
 
+describe('Discover unique properties', function() {
+  let schema;
+  before(function(done) {
+    db.discoverSchema('CUSTOMER', {owner: 'STRONGLOOP'}, function(err, schema_) {
+      schema = schema_;
+      done(err);
+    });
+  });
+  it('should validate unique key for customer', function() {
+    assert.ok(/STRONGLOOP/i.test(schema.options.mysql.schema));
+    assert.strictEqual(schema.options.mysql.table, 'CUSTOMER');
+    assert.strictEqual(schema.properties.email.index.unique, true);
+  });
+});
+
 describe('Discover and handle enum', function() {
   let schema;
   before(function(done) {
@@ -518,7 +533,6 @@ describe('Discover and build models', function() {
     context('with flag treatTINYINT1AsTinyInt = false', function() {
       let models, schema;
       before(discoverAndBuildModels);
-
       it('handles CHAR(1) as Boolean', function() {
         assert(schema.properties.enabled);
         assert.strictEqual(schema.properties.enabled.type, Boolean);
@@ -568,6 +582,11 @@ describe('Discover and build models', function() {
         assert.strictEqual(schema.properties.active.type, Boolean);
       });
 
+      it('handles TINYINT as Boolean', function() {
+        assert(schema.properties.archived);
+        assert.strictEqual(schema.properties.archived.type, Boolean);
+      });
+
       function discoverAndBuildModels(done) {
         db.discoverAndBuildModels('INVENTORY', {
           owner: 'STRONGLOOP',

test/schema.sql

@@ -39,7 +39,8 @@ CREATE TABLE `CUSTOMER` (
   PRIMARY KEY (`ID`)
 ) ENGINE=MyISAM DEFAULT CHARSET=utf8;
 /*!40101 SET character_set_client = @saved_cs_client */;
-
+ALTER TABLE `CUSTOMER`
+ADD COLUMN `email` VARCHAR(100) UNIQUE;
 --
 -- Dumping data for table `CUSTOMER`
 --
@@ -80,6 +81,7 @@ CREATE TABLE `INVENTORY` (
   `TOTAL` int(11) DEFAULT NULL,
   `DETAILS` json DEFAULT NULL,
   `ACTIVE` BOOLEAN DEFAULT TRUE,
+  `ARCHIVED` TINYINT DEFAULT 1,
   `DISABLED` BIT(1) DEFAULT 0,
   `ENABLED` CHAR(1) DEFAULT 'Y',
   PRIMARY KEY (`PRODUCT_ID`,`LOCATION_ID`),