verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 59 Releases Wiki Activity

refs #5858 feat: removeZoneIncludedCheck
gitea/salix/pipeline/pr-dev There was a failure building this commit Details

This commit is contained in:
Javier Segarra 2024-02-12 08:53:55 +01:00
parent a756c3bc3a
commit 995a99c661
3 changed files with 2 additions and 64 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
modules/zone/back/methods/zone/specs/deleteZone.spec.js
View File

@ -47,14 +47,10 @@ describe('zone deletezone()', () => {
it('should NOT delete a zone if is included check', async() => { it('should NOT delete a zone if is included check', async() => {
const tx = await models.Zone.beginTransaction({}); const tx = await models.Zone.beginTransaction({});
let isIncluded = false;
try { try {
const options = {transaction: tx}; const options = {transaction: tx};
const zoneIncludedCheck = await models.ZoneIncludedCheck.find({where: {zoneFk: zoneId}});
isIncluded = zoneIncludedCheck.length > 0;
await models.Zone.deleteZone(ctx, zoneId, options); await models.Zone.deleteZone(ctx, zoneId, options);
} catch (e) { } catch (e) {
expect(isIncluded).toBeTrue();
expect(e).not.toBeNull(); expect(e).not.toBeNull();
await tx.rollback(); await tx.rollback();
} }
@ -71,17 +67,6 @@ describe('zone deletezone()', () => {
const zoneIncludedDeleted = await models.ZoneIncluded.find({where: {zoneFk: zoneId}}, options); const zoneIncludedDeleted = await models.ZoneIncluded.find({where: {zoneFk: zoneId}}, options);
expect(zoneIncludedDeleted.length).toEqual(0); expect(zoneIncludedDeleted.length).toEqual(0);
const zoneIncludedCheck = await models.ZoneIncludedCheck.find({where: {zoneFk: zoneId}}, options);
expect(zoneIncludedCheck.length).toEqual(2);
// Insert
await models.ZoneIncludedCheck.destroyById(zoneIncludedCheck[0].id, options);
// Delete
await models.ZoneIncludedCheck.destroyById(zoneIncludedCheck[1].id, options);
const zoneIncludedCheckDeleted = await models.ZoneIncludedCheck.find({where: {zoneFk: zoneId}}, options);
expect(zoneIncludedCheckDeleted.length).toEqual(0);
await models.Zone.deleteZone(ctx, zoneId, options); await models.Zone.deleteZone(ctx, zoneId, options);

3
modules/zone/back/model-config.json
View File

@ -32,9 +32,6 @@
"ZoneIncluded": { "ZoneIncluded": {
"dataSource": "vn" "dataSource": "vn"
}, },
"ZoneIncludedCheck": {
"dataSource": "vn"
},
"ZoneWarehouse": { "ZoneWarehouse": {
"dataSource": "vn" "dataSource": "vn"
}, },

44
modules/zone/back/models/zone-included-check.json
View File

@ -1,44 +0,0 @@
{
"name": "ZoneIncludedCheck",
"base": "Loggable",
"options": {
"mysql": {
"table": "zoneIncludedCheck"
}
},
"properties": {
"id": {
"id": true,
"type": "number"
},
"zoneFk": {
"type": "int"
},
"geoFk": {
"type": "int"
},
"userFk": {
"type": "int"
},
"action": {
"type": "string"
}
},
"relations": {
"zone": {
"type": "belongsTo",
"model": "Zone",
"foreignKey": "zoneFk"
},
"geo": {
"type": "belongsTo",
"model": "ZoneGeo",
"foreignKey": "geoFk"
},
"user": {
"type": "belongsTo",
"model": "VnUser",
"foreignKey": "userFk"
}
}
}