refactor(protected-areas): avoid cartesian result produced due to wor…

…king on repo
Vizzuality · Sep 16, 2021 · 2ab2fe6 · 2ab2fe6
1 parent fb62e57
commit 2ab2fe6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/api/apps/geoprocessing/src/modules/protected-areas/protected-areas-tiles.service.ts b/api/apps/geoprocessing/src/modules/protected-areas/protected-areas-tiles.service.ts
@@ -1,7 +1,7 @@
 import { Injectable } from '@nestjs/common';
 
 import { InjectRepository } from '@nestjs/typeorm';
-import { Brackets, Repository, getConnection } from 'typeorm';
+import { Brackets, Repository } from 'typeorm';
 import { nominatim2bbox } from '@marxan-geoprocessing/utils/bbox.utils';
 import { TileService } from '@marxan-geoprocessing/modules/tile/tile.service';
 
@@ -38,7 +38,7 @@ export class ProtectedAreasTilesService {
     const inputProjection = 4326;
     const attributes = 'full_name, status, wdpaid, iucn_cat';
     const table = 'wdpa';
-    const query = getConnection().createQueryBuilder();
+    const query = this.areas.manager.createQueryBuilder();
 
     query
       .select(`ST_AsMVT(tile.*, 'layer0', ${extent}, 'mvt_geom')`, 'mvt')

diff --git a/api/apps/geoprocessing/test/tiles/protected-area.fixtures.ts b/api/apps/geoprocessing/test/tiles/protected-area.fixtures.ts
@@ -70,9 +70,8 @@ export const getFixtures = async () => {
         (feature) => feature.properties.full_name === customPa.name,
       );
 
-      // TODO: fixme: original mvt creation produces multiplied layers
-      expect(customFeature.length).toBeGreaterThanOrEqual(1);
-      // expect(features.length).toEqual(10);
+      expect(customFeature.length).toEqual(1);
+      expect(features.length).toEqual(7);
 
       /**
        * TODO maybe: