Check for numbers when parsing variables (#30)

* Ensure numbers are not parsed as JSON

* Bump to v0.5.2

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cda0/terrajs",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "A node interface to Terraform",
   "main": "src/index.js",
   "scripts": {

src/helpers.js

@@ -33,12 +33,13 @@ function parseVariable(key, value) {
     return `'${parsedKey}=${parsedValue.replace(/'/g, '\\\'')}'`;
   }
 
-  if (typeof value === 'boolean') {
-    return `"${parsedKey}=${value}"`;
+  if (typeof value === 'string') {
+    // Double-quotes can prematurely close the string, so they need to be escaped.
+    return `"${parsedKey}=${value.replace(/"/g, '\\"')}"`;
   }
 
-  // Double-quotes can prematurely close the string, so they need to be escaped.
-  return `"${parsedKey}=${value.replace(/"/g, '\\"')}"`;
+  // Booleans and numbers can be returned without parsing.
+  return `"${parsedKey}=${value}"`;
 }
 
 module.exports = {

test/unit/helpers.test.js

@@ -89,6 +89,12 @@ describe('helpers', () => {
         assert.equal(response, '"some_var=West Europe"');
       });
 
+      it('should correctly parse a number variable', () => {
+        const variable = 42;
+        const response = parseVariable('someVar', variable);
+        assert.equal(response, '"some_var=42"');
+      });
+
       it('should correctly parse a boolean variable', () => {
         const variable = true;
         const response = parseVariable('someVar', variable);
@@ -137,6 +143,12 @@ describe('helpers', () => {
         assert.equal(response, '"some_var=West Europe"');
       });
 
+      it('should correctly parse a number variable', () => {
+        const variable = 42;
+        const response = parseVariable('someVar', variable);
+        assert.equal(response, '"some_var=42"');
+      });
+
       it('should correctly parse a boolean variable', () => {
         const variable = true;
         const response = parseVariable('someVar', variable);