Upgrade prettier

And also run prettier as part of the automated tests

Author: ctavan

README.md

@@ -48,12 +48,12 @@ if (Math.random() < abTestRatio) {
 }
 
 const keywords = getenv.array('KEYWORDS');
-keywords.forEach(function(keyword) {
+keywords.forEach(function (keyword) {
   // console.log(keyword);
 });
 
 const primes = getenv.array('PRIMES', 'int');
-primes.forEach(function(prime) {
+primes.forEach(function (prime) {
   // console.log(prime, typeof prime);
 });
 ```

index.js

@@ -28,18 +28,18 @@ function _value(varName, fallback) {
 }
 
 const convert = {
-  string: function(value) {
+  string: function (value) {
     return '' + value;
   },
-  int: function(value) {
+  int: function (value) {
     const isInt = value.match(/^-?\d+$/);
     if (!isInt) {
       throw new Error('GetEnv.NoInteger: ' + value + ' is not an integer.');
     }
 
     return +value;
   },
-  float: function(value) {
+  float: function (value) {
     const isInfinity = +value === Infinity || +value === -Infinity;
     if (isInfinity) {
       throw new Error('GetEnv.Infinity: ' + value + ' is set to +/-Infinity.');
@@ -52,15 +52,16 @@ const convert = {
 
     return +value;
   },
-  bool: function(value) {
-    const isBool = (value || '').toLowerCase() === 'true' || (value || '').toLowerCase() === 'false';
+  bool: function (value) {
+    const isBool =
+      (value || '').toLowerCase() === 'true' || (value || '').toLowerCase() === 'false';
     if (!isBool) {
       throw new Error('GetEnv.NoBoolean: ' + value + ' is not a boolean.');
     }
 
     return (value || '').toLowerCase() === 'true';
   },
-  boolish: function(value) {
+  boolish: function (value) {
     try {
       return convert.bool(value);
     } catch (err) {
@@ -76,7 +77,7 @@ const convert = {
 };
 
 function converter(type) {
-  return function(varName, fallback) {
+  return function (varName, fallback) {
     if (typeof varName == 'string') {
       // default
       const value = _value(varName, fallback);
@@ -90,7 +91,7 @@ function converter(type) {
 
 const getenv = converter('string');
 
-Object.keys(convert).forEach(function(type) {
+Object.keys(convert).forEach(function (type) {
   getenv[type] = converter(type);
 });
 
@@ -131,19 +132,19 @@ getenv.multi = function multi(spec) {
   return result;
 };
 
-getenv.disableFallbacks = function() {
+getenv.disableFallbacks = function () {
   fallbacksDisabled = true;
 };
 
-getenv.enableFallbacks = function() {
+getenv.enableFallbacks = function () {
   fallbacksDisabled = false;
 };
 
-getenv.disableErrors = function() {
+getenv.disableErrors = function () {
   throwError = false;
 };
 
-getenv.enableErrors = function() {
+getenv.enableErrors = function () {
   throwError = true;
 };
 

package.json

@@ -18,15 +18,16 @@
   },
   "main": "index.js",
   "scripts": {
-    "prettier": "prettier --write *.{js,md} **/*.js",
-    "test": "bash -ec 'for F in test/*.js; do echo \"$F\": ; node $F; done;'"
+    "prettier": "prettier --check .",
+    "unit": "bash -ec 'for F in test/*.js; do echo \"$F\": ; node $F; done;'",
+    "test": "npm run prettier && npm run unit"
   },
   "engines": {
     "node": ">=6"
   },
   "dependencies": {},
   "devDependencies": {
-    "prettier": "^1.18.2"
+    "prettier": "^3.5.3"
   },
   "keywords": [
     "env",

test/disableErrors.js

@@ -4,21 +4,21 @@ const getenv = require('../index');
 
 const tests = {};
 
-tests['getenv.disableErrors() should disable any errors'] = function() {
+tests['getenv.disableErrors() should disable any errors'] = function () {
   getenv.disableErrors();
   assert.strictEqual(getenv.string('url', 'http://localhost'), 'http://localhost');
   assert(getenv.string('url'), undefined);
 };
 
-tests['getenv.enableErrors() should enable errors'] = function() {
+tests['getenv.enableErrors() should enable errors'] = function () {
   getenv.enableErrors();
-  assert.throws(function() {
+  assert.throws(function () {
     getenv.string('url');
   });
   assert.strictEqual(getenv.string('url', 'http://localhost'), 'http://localhost');
 };
 
-Object.keys(tests).forEach(function(key) {
+Object.keys(tests).forEach(function (key) {
   console.log('Test: %s', key);
   tests[key]();
 });

test/fallbacks.js

@@ -5,15 +5,15 @@ const getenv = require('../index');
 const tests = {};
 
 // order dependent test
-tests['getenv.disableFallbacks() makes relying on fallbacks an error'] = function() {
+tests['getenv.disableFallbacks() makes relying on fallbacks an error'] = function () {
   getenv.disableFallbacks();
-  assert.throws(function() {
+  assert.throws(function () {
     getenv.string('url', 'http://localhost');
   });
   getenv.enableFallbacks();
 };
 
-Object.keys(tests).forEach(function(key) {
+Object.keys(tests).forEach(function (key) {
   console.log('Test: %s', key);
   tests[key]();
 });