fix: should not lose precision when uncasting string to number (#438)

Author: cyjake

src/data_types.ts

@@ -177,6 +177,7 @@ class INTEGER extends DataType {
     const originValue = value;
     if (value == null || value instanceof Raw) return value;
     if (typeof value === 'string') value = parseInt(value, 10);
+    if (value > Number.MAX_SAFE_INTEGER) return originValue;
     if (Number.isNaN(value)) {
       if (strict) throw new Error(util.format('invalid integer: %s', originValue));
       return originValue;

test/unit/data_types.test.js

@@ -156,6 +156,8 @@ describe('=> DataTypes type casting', function() {
     assert.equal(new INTEGER().uncast(undefined), undefined);
     assert.equal(new INTEGER().uncast('1'), 1);
     assert.equal(new INTEGER().uncast(1), 1);
+    const value = Number.MAX_SAFE_INTEGER.toString() + 1234;
+    assert.equal(new INTEGER().uncast(value), value);
 
     await assert.rejects(async () => {
       new Postgres_DataTypes.INTEGER().uncast('yes?');
@@ -168,7 +170,6 @@ describe('=> DataTypes type casting', function() {
     }, /Error: invalid integer: yes?/);
 
     assert.equal(new INTEGER().uncast('yes?', false), 'yes?');
-
   });
 
   it('STRING', async function() {