typeorm 0.3.11-dev.efb4168 → 0.3.11-dev.f07fb2c

package/driver/aurora-mysql/AuroraMysqlQueryRunner.js

@@ -1226,6 +1226,11 @@ class AuroraMysqlQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 const tableColumn = new TableColumn_1.TableColumn();
                 tableColumn.name = dbColumn["COLUMN_NAME"];
                 tableColumn.type = dbColumn["DATA_TYPE"].toLowerCase();
+                // Unsigned columns are handled differently when it comes to width.
+                // Hence, we need to set the unsigned attribute before we check the width.
+                tableColumn.unsigned = tableColumn.zerofill
+                    ? true
+                    : dbColumn["COLUMN_TYPE"].indexOf("unsigned") !== -1;
                 if (this.driver.withWidthColumnTypes.indexOf(tableColumn.type) !== -1) {
                     const width = dbColumn["COLUMN_TYPE"].substring(dbColumn["COLUMN_TYPE"].indexOf("(") + 1, dbColumn["COLUMN_TYPE"].indexOf(")"));
                     tableColumn.width =
@@ -1272,9 +1277,6 @@ class AuroraMysqlQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 });
                 tableColumn.zerofill =
                     dbColumn["COLUMN_TYPE"].indexOf("zerofill") !== -1;
-                tableColumn.unsigned = tableColumn.zerofill
-                    ? true
-                    : dbColumn["COLUMN_TYPE"].indexOf("unsigned") !== -1;
                 tableColumn.isGenerated =
                     dbColumn["EXTRA"].indexOf("auto_increment") !== -1;
                 if (tableColumn.isGenerated)
@@ -1676,7 +1678,22 @@ class AuroraMysqlQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             this.connection.driver.dataTypeDefaults[column.type] &&
             this.connection.driver.dataTypeDefaults[column.type].width;
         if (defaultWidthForType) {
-            return defaultWidthForType === width;
+            // In MariaDB & MySQL 5.7, the default widths of certain numeric types are 1 less than
+            // the usual defaults when the column is unsigned.
+            // This also applies to Aurora MySQL.
+            const typesWithReducedUnsignedDefault = [
+                "int",
+                "tinyint",
+                "smallint",
+                "mediumint",
+            ];
+            const needsAdjustment = typesWithReducedUnsignedDefault.indexOf(column.type) !== -1;
+            if (column.unsigned && needsAdjustment) {
+                return defaultWidthForType - 1 === width;
+            }
+            else {
+                return defaultWidthForType === width;
+            }
         }
         return false;
     }