refactor: replace _isPrimaryKeyText by _isIntPrimaryKey

Author: HugoPoi

lib/sqlite3.js

@@ -286,7 +286,7 @@ SQLite3.prototype.disconnect = function disconnect(cb) {
 };
 
 SQLite3.prototype.getInsertedId = function(model, info) {
-  if (!this._isPrimaryKeyText(model)) {
+  if (this._isPrimaryKeyInteger(model)) {
     return info.lastID;
   }
 };
@@ -368,13 +368,13 @@ SQLite3.prototype._buildColumnDefinition = function(model, propertyName) {
   return line;
 };
 
-SQLite3.prototype._isPrimaryKeyText = function(model) {
+SQLite3.prototype._isPrimaryKeyInteger = function(model) {
   var properties = this.getModelDefinition(model).properties;
   for (var propertyName in properties) {
     if (!properties.hasOwnProperty(propertyName)) continue;
     var property = this.getModelDefinition(model).properties[propertyName];
-    if (property.id && this._columnDataType(model, propertyName) === 'TEXT') {
-      return true;
+    if (property.id) {
+      return this._columnDataType(model, propertyName) === 'INTEGER';
     }
   }
   return false;

test/sqlite.test.js

@@ -33,10 +33,8 @@ describe('sqlite3 connector', function() {
   });
 
   it('should run migration', function(done) {
-    db.automigrate('PostWithBoolean', function() {
-      db.automigrate('ModelWithPKString', function() {
-        done();
-      });
+    db.automigrate(['PostWithBoolean', 'ModelWithPKString'], function(err) {
+      done(err);
     });
   });