express-generator-typescript 2.5.15 → 2.5.16

package/lib/project-files/package.json

@@ -27,7 +27,7 @@
     "jet-logger": "^2.0.1",
     "jet-paths": "^1.1.0",
     "jet-schema": "^1.4.2",
-    "jet-validators": "^1.0.17",
+    "jet-validators": "^1.1.0",
     "jsonfile": "^6.1.0",
     "module-alias": "^2.2.2",
     "morgan": "^1.10.0"

package/lib/project-files/spec/support/Paths.ts

@@ -1,5 +1,5 @@
 import jetPaths from 'jet-paths';
-import Paths from '@src/routes/common/Paths';
+import Paths from '@src/common/Paths';
 
 
 export default jetPaths(Paths);

package/lib/project-files/spec/support/index.ts

@@ -1,4 +1,5 @@
-import { IReqPropErr } from '@src/routes/common';
+import { Response } from 'supertest';
+import { TParseReqErr } from '@src/routes/common';
 import UserRepo from '@src/repos/UserRepo';
 
 
@@ -6,9 +7,19 @@ import UserRepo from '@src/repos/UserRepo';
                                 Types
 ******************************************************************************/
 
+// Use generics to add properties to 'body'
+export type TRes<T = object> = Omit<Response, 'body'> & {
+  body: T & { error?: string | IErrObj },
+};
+
+interface IErrObj {
+  message: string;
+  [key: string]: unknown;
+}
+
 export interface IValidationErr {
   message: string;
-  parameters: IReqPropErr[];
+  parameters: TParseReqErr[];
 }
 
 

package/lib/project-files/spec/tests/users.spec.ts

@@ -12,7 +12,8 @@ import HttpStatusCodes from '@src/common/HttpStatusCodes';
 import { ValidationErr } from '@src/common/route-errors';
 
 import Paths from 'spec/support/Paths';
-import { cleanDatabase, IValidationErr } from 'spec/support';
+import { cleanDatabase, IValidationErr, TRes } from 'spec/support';
+import { isObject } from 'jet-validators';
 
 
 /******************************************************************************
@@ -30,7 +31,7 @@ const DB_USERS = [
 // database
 const compareUserArrays = customDeepCompare({
   onlyCompareProps: ['name', 'email'],
-})
+});
 
 
 /******************************************************************************
@@ -59,7 +60,7 @@ describe('UserRouter', () => {
     `of "${HttpStatusCodes.OK}" if the request was successful.`, done => {
       agent
         .get(Paths.Users.Get)
-        .end((_, res) => {
+        .end((_, res: TRes<{ users: IUser[]}>) => {
           expect(res.status).toBe(HttpStatusCodes.OK);
           expect(compareUserArrays(res.body.users, DB_USERS)).toBeTruthy();
           done();
@@ -90,11 +91,12 @@ describe('UserRouter', () => {
       agent
         .post(Paths.Users.Add)
         .send({ user: null })
-        .end((_, res) => {
+        .end((_, res: TRes) => {
           expect(res.status).toBe(HttpStatusCodes.BAD_REQUEST);
-          const errorObj = parseJson<IValidationErr>(res.body.error);
+          const errorObj = parseJson<IValidationErr>(res.body.error),
+            parseErr = errorObj.parameters[0];
           expect(errorObj.message).toBe(ValidationErr.MSG);
-          expect(errorObj.parameters[0].prop).toBe('user');
+          expect(isObject(parseErr) && parseErr.prop === 'user').toBeTruthy();
           done();
         });
     });
@@ -124,11 +126,12 @@ describe('UserRouter', () => {
       agent
         .put(Paths.Users.Update)
         .send({ user: null })
-        .end((_, res) => {
+        .end((_, res: TRes) => {
           expect(res.status).toBe(HttpStatusCodes.BAD_REQUEST);
-          const errorObj = parseJson<IValidationErr>(res.body.error);
+          const errorObj = parseJson<IValidationErr>(res.body.error),
+            parseErr = errorObj.parameters[0];
           expect(errorObj.message).toBe(ValidationErr.MSG);
-          expect(errorObj.parameters[0].prop).toBe('user');
+          expect(isObject(parseErr) && parseErr.prop === 'user').toBeTruthy();
           done();
         });
     });
@@ -141,7 +144,7 @@ describe('UserRouter', () => {
       agent
         .put(Paths.Users.Update)
         .send({ user })
-        .end((_, res) => {
+        .end((_, res: TRes) => {
           expect(res.status).toBe(HttpStatusCodes.NOT_FOUND);
           expect(res.body.error).toBe(USER_NOT_FOUND_ERR);
           done();
@@ -172,7 +175,7 @@ describe('UserRouter', () => {
     `"${HttpStatusCodes.NOT_FOUND}" if the id was not found.`, done => {
       agent
         .delete(getPath(-1))
-        .end((_, res) => {
+        .end((_, res: TRes) => {
           expect(res.status).toBe(HttpStatusCodes.NOT_FOUND);
           expect(res.body.error).toBe(USER_NOT_FOUND_ERR);
           done();

package/lib/project-files/src/common/ENV.ts

@@ -5,10 +5,17 @@ import { NodeEnvs } from './constants';
 
 
 /******************************************************************************
-                            Export default
+                                 Setup
 ******************************************************************************/
 
-export default jetEnv({
+const ENV = jetEnv({
   NodeEnv: isEnumVal(NodeEnvs),
   Port: num,
 });
+
+
+/******************************************************************************
+                            Export default
+******************************************************************************/
+
+export default ENV;

package/lib/project-files/src/common/constants.ts

@@ -3,8 +3,9 @@
                               Enums
 ******************************************************************************/
 
+// NOTE: These need to match the names of your ".env" files
 export enum NodeEnvs {
   Dev = 'development',
   Test = 'test',
   Production = 'production'
-}
+}

package/lib/project-files/src/repos/MockOrm.ts

@@ -11,8 +11,8 @@ import { IUser } from '@src/models/User';
 
 const DB_FILE_NAME = (
   ENV.NodeEnv === NodeEnvs.Test 
-  ? 'database.test.json' 
-  : 'database.json'
+    ? 'database.test.json' 
+    : 'database.json'
 );
 
 

package/lib/project-files/src/repos/database.test.json

@@ -1 +1 @@
-{"users":[{"id":697609403164,"name":"Bill","created":"2025-02-13T19:25:15.288Z","email":"sean.maxwell@gmail.com"},{"id":387041929717,"name":"John Smith","created":"2025-02-13T19:25:15.288Z","email":"john.smith@gmail.com"},{"id":465077000925,"name":"Gordan Freeman","created":"2025-02-13T19:25:15.288Z","email":"gordan.freeman@gmail.com"}]}
+{"users":[{"id":833335167307,"name":"Bill","created":"2025-02-21T15:41:21.593Z","email":"sean.maxwell@gmail.com"},{"id":721615734671,"name":"John Smith","created":"2025-02-21T15:41:21.593Z","email":"john.smith@gmail.com"},{"id":531901228132,"name":"Gordan Freeman","created":"2025-02-21T15:41:21.593Z","email":"gordan.freeman@gmail.com"},{"id":811228870992,"name":"a","created":"2025-02-21T15:41:21.594Z","email":"a@a.com"}]}

package/lib/project-files/src/routes/UserRoutes.ts

@@ -5,7 +5,6 @@ import HttpStatusCodes from '@src/common/HttpStatusCodes';
 import UserService from '@src/services/UserService';
 import User from '@src/models/User';
 
-
 import { parseReq, IReq, IRes } from './common';
 
 

package/lib/project-files/src/routes/common/index.ts

@@ -13,11 +13,11 @@ type TRecord = Record<string, unknown>;
 export type IReq = Request<TRecord, void, TRecord, TRecord>;
 export type IRes = Response<unknown, TRecord>;
 
-export interface IReqPropErr {
-  prop: string;
-  value: unknown;
-  moreInfo?: string;
-}
+export type TParseReqErr = {
+  prop: string,
+  value: unknown,
+  moreInfo?: string,
+} | string;
 
 
 /******************************************************************************
@@ -29,34 +29,50 @@ export interface IReqPropErr {
  */
 export function parseReq<U extends TSchema>(schema: U) {
   return (arg: unknown) => {
-    // Don't alter original object
+    // Don't alter original object (shallow copy is good enough)
     if (isObject(arg)) {
       arg = { ...arg };
     }
-    // Error callback
-    const errArr: IReqPropErr[] = [];
-    const errCb = (
-      prop = 'undefined',
-      value?: unknown,
-      caughtErr?: unknown,
-    ) => {
-      const err: IReqPropErr = { prop, value };
-      if (caughtErr !== undefined) {
-        let moreInfo;
-        if (!isString(caughtErr)) {
-          moreInfo = JSON.stringify(caughtErr);
-        } else {
-          moreInfo = caughtErr;
-        }
-        err.moreInfo = moreInfo;
-      }
-      errArr.push(err);
-    };
-    // Return
+    // Setup error callback
+    const errArr: TParseReqErr[] = [],
+      errCb = setupErrorCb(errArr);
+    // Run Tests
     const retVal = parseObject<U>(schema, errCb)(arg);
     if (errArr.length > 0) {
       throw new ValidationErr(errArr);
     }
+    // Return
     return retVal;
   };
 }
+
+/**
+ * Setup the error callback function for when "parseReq" fires and error.
+ */
+function setupErrorCb(errArr: TParseReqErr[]) {
+  return function (
+    prop = 'undefined',
+    value?: unknown,
+    caughtErr?: unknown,
+  ) {
+    // Initialize err
+    let err: TParseReqErr;
+    if (arguments.length === 1) {
+      err = prop;
+    } else {
+      err = { prop, value };
+    }
+    // Check if there's a "caught error"
+    if (isObject(err) && caughtErr !== undefined) {
+      let moreInfo;
+      if (!isString(caughtErr)) {
+        moreInfo = JSON.stringify(caughtErr);
+      } else {
+        moreInfo = caughtErr;
+      }
+      err.moreInfo = moreInfo;
+    }
+    // Add error to array
+    errArr.push(err);
+  };
+}

package/lib/project-files/src/routes/index.ts

@@ -1,6 +1,6 @@
 import { Router } from 'express';
 
-import Paths from './common/Paths';
+import Paths from '../common/Paths';
 import UserRoutes from './UserRoutes';
 
 

package/lib/project-files/src/server.ts

@@ -8,7 +8,7 @@ import 'express-async-errors';
 
 import BaseRouter from '@src/routes';
 
-import Paths from '@src/routes/common/Paths';
+import Paths from '@src/common/Paths';
 import ENV from '@src/common/ENV';
 import HttpStatusCodes from '@src/common/HttpStatusCodes';
 import { RouteError } from '@src/common/route-errors';
@@ -16,25 +16,25 @@ import { NodeEnvs } from '@src/common/constants';
 
 
 /******************************************************************************
-                                Variables
+                                Setup
 ******************************************************************************/
 
 const app = express();
 
 
-// **** Setup
+// **** Middleware **** //
 
 // Basic middleware
 app.use(express.json());
 app.use(express.urlencoded({extended: true}));
 
 // Show routes called in console during development
-if (ENV.NodeEnv === NodeEnvs.Dev.valueOf()) {
+if (ENV.NodeEnv === NodeEnvs.Dev) {
   app.use(morgan('dev'));
 }
 
 // Security
-if (ENV.NodeEnv === NodeEnvs.Production.valueOf()) {
+if (ENV.NodeEnv === NodeEnvs.Production) {
   app.use(helmet());
 }
 
@@ -55,7 +55,7 @@ app.use((err: Error, _: Request, res: Response, next: NextFunction) => {
 });
 
 
-// **** Front-End Content
+// **** FrontEnd Content **** //
 
 // Set views directory (html)
 const viewsDir = path.join(__dirname, 'views');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "express-generator-typescript",
-  "version": "2.5.15",
+  "version": "2.5.16",
   "description": "Generate new Express applications similar to express-generate which but sets it up to use TypeScript instead",
   "scripts": {
     "express-generator-typescript": "node bin/cli.js",