Migration:generate issue with onUpdate using mariadb 10.4

src/driver/mysql/MysqlQueryRunner.ts

@@ -1321,7 +1321,9 @@ export class MysqlQueryRunner extends BaseQueryRunner implements QueryRunner {
                     }
 
                     if (dbColumn["EXTRA"].indexOf("on update") !== -1) {
-                        tableColumn.onUpdate = dbColumn["EXTRA"].substring(dbColumn["EXTRA"].indexOf("on update") + 10);
+                        // New versions of MariaDB return expressions in lowercase.  We need to set it in
+                        // uppercase so the comparison in MysqlDriver#compareExtraValues does not fail.
+                        tableColumn.onUpdate = dbColumn["EXTRA"].substring(dbColumn["EXTRA"].indexOf("on update") + 10).toUpperCase();
                     }
 
                     if (dbColumn["GENERATION_EXPRESSION"]) {

test/github-issues/6714/entity/session.ts

@@ -0,0 +1,17 @@
+import { Entity, PrimaryGeneratedColumn, Column } from "../../../../src";
+
+@Entity({ name: "Session" })
+export class Session {
+
+    @PrimaryGeneratedColumn()
+    id?: number;
+
+    @Column({
+        type: "timestamp",
+        precision: 3,
+        default: () => "CURRENT_TIMESTAMP(3)",
+        onUpdate: "CURRENT_TIMESTAMP(3)",
+    })
+    ts: Date;
+
+}

test/github-issues/6714/entity/sessionchanged.ts

@@ -0,0 +1,15 @@
+import { Entity, PrimaryGeneratedColumn, Column } from "../../../../src";
+
+@Entity({ name: "Session" })
+export class Session {
+    @PrimaryGeneratedColumn()
+    id?: number;
+
+    @Column({
+        type: "timestamp",
+        precision: 4,
+        default: () => "CURRENT_TIMESTAMP(4)",
+        onUpdate: "CURRENT_TIMESTAMP(4)",
+    })
+    ts: Date;
+}

test/github-issues/6714/issue-6714.ts

@@ -0,0 +1,59 @@
+import "reflect-metadata";
+import {
+    createTestingConnections,
+    closeTestingConnections,
+    reloadTestingDatabases,
+} from "../../utils/test-utils";
+import { Connection } from "../../../src/connection/Connection";
+import { expect } from "chai";
+import { Session as baseEntity } from "./entity/session";
+import { Session as changedEntity } from "./entity/sessionchanged";
+
+describe("github issues > #6714 Migration:generate issue with onUpdate using mariadb 10.4", () => {
+    it("dont change anything", async () => {
+        let connections: Connection[];
+        connections = await createTestingConnections({
+            entities: [baseEntity],
+            schemaCreate: false,
+            dropSchema: true,
+            enabledDrivers: ["mariadb"],
+        });
+        await reloadTestingDatabases(connections);
+        await Promise.all(
+            connections.map(async (connection) => {
+                const schemaBuilder = connection.driver.createSchemaBuilder();
+                const syncQueries = await schemaBuilder.log();
+                expect(syncQueries.downQueries).to.be.eql([]);
+                expect(syncQueries.upQueries).to.be.eql([]);
+            })
+        );
+        await closeTestingConnections(connections);
+    });
+    it("recognizing on update changes", async () => {
+        // this connection create database with a Session entity
+        const baseConnections = await createTestingConnections({
+            entities: [baseEntity],
+            schemaCreate: true, // create the database
+            dropSchema: true,
+            enabledDrivers: ["mariadb"],
+        });
+        // this connection change Session entity on update value
+        const connections = await createTestingConnections({
+            entities: [changedEntity],
+            schemaCreate: false, // don't change the entity
+            dropSchema: false,
+            enabledDrivers: ["mariadb"],
+            name: "test",
+        });
+        await Promise.all(
+            connections.map(async (connection) => {
+                const schemaBuilder = connection.driver.createSchemaBuilder();
+                const syncQueries = await schemaBuilder.log();
+                expect(syncQueries.downQueries.length).not.to.be.eql(0);
+                expect(syncQueries.upQueries.length).not.to.be.eql(0);
+            })
+        );
+        await closeTestingConnections(baseConnections);
+        await closeTestingConnections(connections);
+    });
+});