@stanlemon/server 0.2.1 → 0.2.5

package/jest.config.js

@@ -0,0 +1 @@
+export { default } from "@stanlemon/webdev/jest.config.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stanlemon/server",
-  "version": "0.2.1",
+  "version": "0.2.5",
   "description": "A basic express web server setup.",
   "author": "Stan Lemon <stanlemon@users.noreply.github.com>",
   "license": "MIT",
@@ -13,21 +13,26 @@
   "scripts": {
     "start": "NODE_ENV=development nodemon ./app.js",
     "lint": "eslint --ext js,jsx,ts,tsx ./",
-    "lint:format": "eslint --fix --ext js,jsx,ts,tsx ./"
+    "lint:format": "eslint --fix --ext js,jsx,ts,tsx ./",
+    "test": "jest --detectOpenHandles",
+    "test:coverage": "jest --detectOpenHandles --coverage",
+    "test:watch": "jest --detectOpenHandles --watch"
   },
   "dependencies": {
+    "@stanlemon/webdev": "*",
     "compression": "^1.7.4",
     "dotenv": "16.0.0",
-    "express": "^4.17.3",
-    "express-rate-limit": "^6.3.0",
+    "express": "^4.18.1",
+    "express-rate-limit": "^6.4.0",
     "helmet": "^5.0.2",
-    "http-proxy-middleware": "^2.0.4",
+    "http-proxy-middleware": "^2.0.6",
     "joi": "^17.6.0",
-    "lodash-es": "^4.17.21",
+    "lodash": "^4.17.21",
     "morgan": "^1.10.0"
   },
   "devDependencies": {
     "@stanlemon/eslint-config": "*",
-    "nodemon": "^2.0.15"
+    "@types/lodash": "^4.14.182",
+    "nodemon": "^2.0.16"
   }
 }

package/src/asyncJsonHandler.js

@@ -37,10 +37,7 @@ export function asyncJsonHandler(fn) {
         return;
       }
 
-      if (
-        process.env.NODE_ENV === "development" ||
-        process.env.NODE_ENV === "test"
-      ) {
+      if (process.env.NODE_ENV !== "production") {
         // eslint-disable-next-line no-console
         console.error(ex);
 

package/src/convertCase.js

@@ -1,4 +1,4 @@
-import { isObject, isArray, isString } from "lodash-es";
+import { isObject, isString } from "lodash-es";
 
 export default function convertCase(obj, me, convert) {
   if (
@@ -8,9 +8,9 @@ export default function convertCase(obj, me, convert) {
     )
   ) {
     return new Date(obj);
-  } else if (isArray(obj)) {
+  } else if (Array.isArray(obj)) {
     return obj.map((i) => me(i));
-  } else if (isObject(obj) && !isArray(obj)) {
+  } else if (isObject(obj) && !Array.isArray(obj)) {
     const n = {};
     Object.keys(obj).forEach((k) => (n[convert(k)] = me(obj[k])));
     return n;

package/src/convertCase.test.js

@@ -0,0 +1,30 @@
+import convertCase from "./convertCase";
+import { snakeCase } from "lodash-es";
+
+describe("convertCase()", () => {
+  const convert = (obj) => convertCase(obj, convert, snakeCase);
+
+  it("converts object keys", async () => {
+    expect(convert({ fooBar: true }, convertCase, snakeCase)).toEqual({
+      foo_bar: true,
+    });
+  });
+
+  it("converts an array of object keys", async () => {
+    expect(convert([{ fooBar: true }], convertCase, snakeCase)).toEqual([
+      {
+        foo_bar: true,
+      },
+    ]);
+  });
+
+  it("does not convert strings", async () => {
+    expect(convert("hereIsAString", convertCase, snakeCase)).toEqual(
+      "hereIsAString"
+    );
+  });
+
+  it("does not convert anything else", async () => {
+    expect(convert(true, convertCase, snakeCase)).toEqual(true);
+  });
+});

package/src/createAppServer.js

@@ -33,7 +33,11 @@ export default function createAppServer(options) {
     app.use(helmet());
   }
 
-  if (webpack !== false && process.env.NODE_ENV !== "production") {
+  if (
+    webpack !== false &&
+    process.env.NODE_ENV !== "production" &&
+    process.env.NODE_ENV !== "test"
+  ) {
     app.get(
       "/*.js",
       createProxyMiddleware({
@@ -68,7 +72,8 @@ export default function createAppServer(options) {
     });
   });
 
-  if (start) {
+  // If we're set to start. Btw we never start in test.
+  if (start && process.env.NODE_ENV !== "test") {
     const server = app.listen(port);
 
     /* eslint-disable no-console */

package/src/formatInput.js

@@ -1,8 +1,8 @@
-import { isArray, snakeCase } from "lodash-es";
+import { snakeCase } from "lodash-es";
 import convertCase from "./convertCase.js";
 
 export default function formatInput(obj) {
-  if (isArray(obj)) {
+  if (Array.isArray(obj)) {
     return obj.map((v) => formatInput(v));
   }
   return convertCase(obj, formatInput, snakeCase);

package/src/formatInput.test.js

@@ -0,0 +1,31 @@
+import formatInput from "./formatInput";
+
+describe("formatInput()", () => {
+  it("formats object keys", async () => {
+    expect(formatInput({ fooBar: true })).toEqual({
+      foo_bar: true,
+    });
+  });
+
+  it("formats an array of object keys", async () => {
+    expect(formatInput([{ fooBar: true }])).toEqual([
+      {
+        foo_bar: true,
+      },
+    ]);
+  });
+
+  it("does not formats strings", async () => {
+    expect(formatInput("hereIsAString")).toEqual("hereIsAString");
+  });
+
+  it("does not formats anything else", async () => {
+    expect(formatInput(true)).toEqual(true);
+  });
+
+  it("does not change already formatted object keys", async () => {
+    expect(formatInput({ foo_bar: true })).toEqual({
+      foo_bar: true,
+    });
+  });
+});

package/src/formatOutput.js

@@ -1,11 +1,4 @@
-import {
-  camelCase,
-  isPlainObject,
-  isArray,
-  isDate,
-  isEmpty,
-  omit,
-} from "lodash-es";
+import { camelCase, isPlainObject, isDate, isEmpty, omit } from "lodash-es";
 import convertCase from "./convertCase.js";
 
 export default function formatOutput(o, omittedFields = []) {
@@ -14,7 +7,7 @@ export default function formatOutput(o, omittedFields = []) {
   if (isDate(obj)) {
     return obj.toISOString();
   }
-  if (isArray(obj)) {
+  if (Array.isArray(obj)) {
     return obj.map((v) => formatOutput(v));
   }
   return convertCase(obj, formatOutput, camelCase);

package/src/formatOutput.test.js

@@ -0,0 +1,31 @@
+import formatOutput from "./formatOutput";
+
+describe("formatOutput()", () => {
+  it("formats object keys", async () => {
+    expect(formatOutput({ foo_bar: true })).toEqual({
+      fooBar: true,
+    });
+  });
+
+  it("formats an array of object keys", async () => {
+    expect(formatOutput([{ foo_bar: true }])).toEqual([
+      {
+        fooBar: true,
+      },
+    ]);
+  });
+
+  it("does not formats strings", async () => {
+    expect(formatOutput("hereIsAString")).toEqual("hereIsAString");
+  });
+
+  it("does not formats anything else", async () => {
+    expect(formatOutput(true)).toEqual(true);
+  });
+
+  it("does not change already formatted object keys", async () => {
+    expect(formatOutput({ fooBar: true })).toEqual({
+      fooBar: true,
+    });
+  });
+});