Merge pull request #22 from stephenlacy/allcaps

Add support for all caps boolean values

Author: ctavan

index.js

@@ -53,12 +53,12 @@ const convert = {
     return +value;
   },
   bool: function(value) {
-    const isBool = value === 'true' || value === 'false';
+    const isBool = (value || '').toLowerCase() === 'true' || (value || '').toLowerCase() === 'false';
     if (!isBool) {
       throw new Error('GetEnv.NoBoolean: ' + value + ' is not a boolean.');
     }
 
-    return value === 'true';
+    return (value || '').toLowerCase() === 'true';
   },
   boolish: function(value) {
     try {

test/getenv.js

@@ -13,7 +13,9 @@ process.env.TEST_GETENV_FLOAT2 = '0.0';
 process.env.TEST_GETENV_INFINITY1 = Infinity;
 process.env.TEST_GETENV_INFINITY2 = -Infinity;
 process.env.TEST_GETENV_FALSE = 'false';
+process.env.TEST_GETENV_FALSE2 = 'FALSE';
 process.env.TEST_GETENV_TRUE = 'true';
+process.env.TEST_GETENV_TRUE2 = 'TRUE';
 process.env.TEST_GETENV_NOT_REALLY_TRUE = '1';
 process.env.TEST_GETENV_NOT_REALLY_FALSE = '0';
 process.env.TEST_GETENV_WRONG_NUMBER_INPUT = '3 test';
@@ -202,10 +204,18 @@ tests['getenv.bool() valid input'] = function() {
       varName: 'TEST_GETENV_FALSE',
       expected: false,
     },
+    {
+      varName: 'TEST_GETENV_FALSE2',
+      expected: false,
+    },
     {
       varName: 'TEST_GETENV_TRUE',
       expected: true,
     },
+    {
+      varName: 'TEST_GETENV_TRUE2',
+      expected: true,
+    },
   ];
 
   data.forEach(function(item) {