fix: do not return empty array for missing projections

geops · Mar 21, 2020 · 26db797 · 26db797
1 parent 89012e2
commit 26db797
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/project.js b/src/project.js
@@ -107,8 +107,9 @@ function project(location = {}, targetAltitude, previousResult) {
     const { transitionForestType, transitionAltitudinalZone, ...tl } = location;
     tl.forestType = transitionForestType;
     tl.altitudinalZone = transitionAltitudinalZone;
-    const { options, projections: tp } = project(tl, targetAltitude);
-    Object.entries(options).forEach(([k, v]) => {
+    const transition = project(tl, targetAltitude);
+    const tp = transition.projections || [];
+    Object.entries(transition.options).forEach(([k, v]) => {
       if (fieldsConcat.includes(k)) {
         result.options[k] = concat(result.options[k], v);
       }
@@ -126,6 +127,10 @@ function project(location = {}, targetAltitude, previousResult) {
       .map(ft => ft.code);
   }
 
+  if (!previousResult && result.projections.length === 0) {
+    delete result.projections;
+  }
+
   return result;
 }
 

diff --git a/test/project.test.js b/test/project.test.js
@@ -239,7 +239,7 @@ describe('valid projections', () => {
         },
         '50',
       ).projections,
-    ).toStrictEqual([]);
+    ).toStrictEqual(undefined);
   });
 
   test('multi altitudinal zone projection', () => {
@@ -291,7 +291,7 @@ describe('valid projections', () => {
         },
         '30',
       ).projections,
-    ).toStrictEqual([]);
+    ).toStrictEqual(undefined);
   });
 
   test('projections based on fixtures', () => {