feat(describeTable): support string length for mssql (#9896)

lib/dialects/mssql/query.js

@@ -211,6 +211,14 @@ class Query extends AbstractQuery {
           autoIncrement: _result.IsIdentity === 1,
           autoIncrement: _result.IsIdentity === 1,
           comment: _result.Comment
           comment: _result.Comment
         };
         };
+
+        if (
+          result[_result.Name].type.includes('VARCHAR')
+          && _result.Length
+        ) {
+          result[_result.Name].type += `(${_result.Length})`;
+        }
+
       }
       }
     } else if (this.isShowIndexesQuery()) {
     } else if (this.isShowIndexesQuery()) {
       result = this.handleShowIndexesQuery(data);
       result = this.handleShowIndexesQuery(data);

test/integration/query-interface/describeTable.test.js

@@ -50,7 +50,7 @@ describe(Support.getTestDialectTeaser('QueryInterface'), () => {
               assertVal = 'CHARACTER VARYING(255)';
               assertVal = 'CHARACTER VARYING(255)';
               break;
               break;
             case 'mssql':
             case 'mssql':
-              assertVal = 'NVARCHAR';
+              assertVal = 'NVARCHAR(255)';
               break;
               break;
           }
           }
           expect(username.type).to.equal(assertVal);
           expect(username.type).to.equal(assertVal);