Merge pull request #2 from bboure/fixes

Fix ArrayNode for empty arrays, remove type: module, default empty object when params/resolvers not provided

package.json

@@ -3,7 +3,6 @@
   "version": "1.1.0",
   "description": "Library that resolves AWS Cloudformation templates with exact values",
   "main": "src/index.js",
-  "type": "module",
   "scripts": {
     "test": "mocha --recursive",
     "coverage": "nyc --all --reporter cobertura --reporter text --report-dir ./coverage npm run test",

src/convertNode.js

@@ -22,7 +22,7 @@ const azMapping = require('./AZMap.json');
 
 const convertNode = (node, nodeAccessor, srcObj, params, convRoot, enableVerboseLogging) => {
     // keeping backward compatibility because of typo in previos versions
-    const refResolvers = params.RefResolvers || params.RefResolevers; 
+    const refResolvers = params.RefResolvers || params.RefResolevers || {}; 
 
     switch (nodeAccessor.key) {
         case "Fn::FindInMap":
@@ -55,7 +55,7 @@ const convertNode = (node, nodeAccessor, srcObj, params, convRoot, enableVerbose
         case "Ref":
             return new RefNode(node, nodeAccessor, enableVerboseLogging, refResolvers);
         case "Fn::GetAtt":
-            return new FnGetAttNode(node, nodeAccessor, enableVerboseLogging, params["Fn::GetAttResolvers"]);
+            return new FnGetAttNode(node, nodeAccessor, enableVerboseLogging, params["Fn::GetAttResolvers"] || {});
     }
 
     if( _.isArray(nodeAccessor.node)){

src/nodeEvaluator.js

@@ -4,7 +4,7 @@ const { convertNode } = require("./convertNode");
 class NodeEvaluator {
     constructor(srcObj, params, enableVerboseLogging){
         this.srcObj = srcObj;
-        this.params = params;
+        this.params = params || {};
         this.enableVerboseLogging = enableVerboseLogging;
     }
 

src/nodeTypes/ArrayNode.js

@@ -27,21 +27,18 @@ class ArrayNode extends Node {
         super.log("Eval: ", this.nodeAccessor.path.join('/'));
         if(this.isLeaf){
             super.log("Leaf: ",  this.node, this.nodeAccessor.path.join('/'));
-            console.warn("Invalid Array: has no items, how to evaluate??");
-            throw "Invalid Array: has no items, how to evaluate??";
-            return ""; // Simple value (number, string)
-        } else {
-            const result = []
-            this.directDependencies.forEach( (dep) => {
-                const depRes = dep.evaluate();
-                result[dep.nodeAccessor.key] = depRes;
-            });
-
-            super.log("Array evaluated: ");
-            super.log(result);
-
-            return this.evaluateResultedArray(result);
         }
+
+        const result = []
+        this.directDependencies.forEach( (dep) => {
+            const depRes = dep.evaluate();
+            result[dep.nodeAccessor.key] = depRes;
+        });
+
+        super.log("Array evaluated: ");
+        super.log(result);
+
+        return this.evaluateResultedArray(result);
     }
 }
 

src/nodeTypes/Node.js

@@ -6,9 +6,9 @@ class Node {
         this.enableVerboseLogging = enableVerboseLogging;
     }
 
-    log(str) {
+    log(...str) {
         if(this.enableVerboseLogging){
-            console.log(str);
+            console.log(...str);
         }
     }
 

test/nodeTypes/ArrayNode.test.js

@@ -32,5 +32,9 @@ describe('ArrayNode', () => {
     expect(actual).to.deep.equal(["item0", nestedArray , "item2"])
   });
 
-  // TODO: testcase when array is empty
+  it('evaluate empty arrays', () => {
+    const actual = target.evaluate();
+
+    expect(actual).to.deep.equal([])
+  });
 });