Skip to content

Commit 748fe22

Browse files
aaqilnizsamarpanB
authored andcommitted
fix: return generated: false if the connector doesnot return it
Signed-off-by: Muhammad Aaqil <[email protected]>
1 parent 901486f commit 748fe22

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

lib/datasource.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1673,7 +1673,7 @@ DataSource.prototype.discoverSchemas = function(tableName, options, cb) {
16731673
length: item.dataLength,
16741674
precision: item.dataPrecision,
16751675
scale: item.dataScale,
1676-
generated: item.generated,
1676+
generated: item.generated || false,
16771677
};
16781678
if (pks[item.columnName]) {
16791679
schema.properties[propName].id = pks[item.columnName];
@@ -1685,7 +1685,7 @@ DataSource.prototype.discoverSchemas = function(tableName, options, cb) {
16851685
dataPrecision: item.dataPrecision,
16861686
dataScale: item.dataScale,
16871687
nullable: item.nullable,
1688-
generated: item.generated,
1688+
generated: item.generated || false,
16891689
};
16901690
// merge connector-specific properties
16911691
if (item[dbType]) {
@@ -1837,7 +1837,7 @@ DataSource.prototype.discoverSchemasSync = function(modelName, options) {
18371837
length: item.dataLength,
18381838
precision: item.dataPrecision,
18391839
scale: item.dataScale,
1840-
generated: item.generated,
1840+
generated: item.generated || false,
18411841
};
18421842

18431843
if (pks[item.columnName]) {
@@ -1850,7 +1850,7 @@ DataSource.prototype.discoverSchemasSync = function(modelName, options) {
18501850
dataPrecision: item.dataPrecision,
18511851
dataScale: item.dataScale,
18521852
nullable: i.nullable,
1853-
generated: i.generated,
1853+
generated: i.generated || false,
18541854
};
18551855
});
18561856

test/discovery.test.js

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ describe('Memory connector with mocked discovery', function() {
4545
dataPrecision: null,
4646
dataScale: null,
4747
nullable: 0,
48-
generated: undefined,
48+
generated: false,
4949
},
5050
{
5151
owner: 'STRONGLOOP',
@@ -56,7 +56,7 @@ describe('Memory connector with mocked discovery', function() {
5656
dataPrecision: 10,
5757
dataScale: 0,
5858
nullable: 1,
59-
generated: undefined,
59+
generated: false,
6060
},
6161
{
6262
owner: 'STRONGLOOP',
@@ -67,7 +67,7 @@ describe('Memory connector with mocked discovery', function() {
6767
dataPrecision: 10,
6868
dataScale: 0,
6969
nullable: 1,
70-
generated: undefined,
70+
generated: false,
7171
}];
7272

7373
ds.discoverModelProperties = function(modelName, options, cb) {
@@ -234,13 +234,13 @@ describe('Memory connector with mocked discovery', function() {
234234
dataScale: 0,
235235
dataType: 'int',
236236
nullable: 1,
237-
generated: undefined,
237+
generated: false,
238238
},
239239
precision: 10,
240240
required: false,
241241
scale: 0,
242242
type: undefined,
243-
generated: undefined,
243+
generated: false,
244244
},
245245
locationId: {
246246
length: 20,
@@ -254,13 +254,13 @@ describe('Memory connector with mocked discovery', function() {
254254
dataScale: null,
255255
dataType: 'varchar',
256256
nullable: 0,
257-
generated: undefined,
257+
generated: false,
258258
},
259259
precision: null,
260260
required: true,
261261
scale: null,
262262
type: undefined,
263-
generated: undefined,
263+
generated: false,
264264
},
265265
productId: {
266266
length: 20,
@@ -294,13 +294,13 @@ describe('Memory connector with mocked discovery', function() {
294294
dataScale: 0,
295295
dataType: 'int',
296296
nullable: 1,
297-
generated: undefined,
297+
generated: false,
298298
},
299299
precision: 10,
300300
required: false,
301301
scale: 0,
302302
type: undefined,
303-
generated: undefined,
303+
generated: false,
304304
},
305305
},
306306
};
@@ -418,7 +418,7 @@ describe('discoverModelProperties', function() {
418418
dataPrecision: null,
419419
dataScale: null,
420420
nullable: 0,
421-
generated: undefined,
421+
generated: false,
422422
},
423423
{
424424
owner: 'STRONGLOOP',
@@ -429,7 +429,7 @@ describe('discoverModelProperties', function() {
429429
dataPrecision: null,
430430
dataScale: null,
431431
nullable: 0,
432-
generated: undefined,
432+
generated: false,
433433
},
434434
{
435435
owner: 'STRONGLOOP',

0 commit comments

Comments
 (0)