allow opts to be undefined

ludicroushq · Aug 25, 2019 · 3100f11 · 3100f11
1 parent 9f54b08
commit 3100f11
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 9 deletions.
diff --git a/lib/route.js b/lib/route.js
@@ -12,11 +12,12 @@ class Route {
         this.disableErrorHandler = false;
         this.req = req;
         this.res = res;
-        if (!this.disableRequestLogger && opts.disableRequestLogger) {
-            this.disableRequestLogger = opts.disableRequestLogger;
+        const options = opts || {};
+        if (!this.disableRequestLogger && options.disableRequestLogger) {
+            this.disableRequestLogger = options.disableRequestLogger;
         }
-        if (!this.disableErrorHandler && opts.disableErrorHandler) {
-            this.disableErrorHandler = opts.disableErrorHandler;
+        if (!this.disableErrorHandler && options.disableErrorHandler) {
+            this.disableErrorHandler = options.disableErrorHandler;
         }
     }
     _getInternalPlugins() {

diff --git a/src/route.ts b/src/route.ts
@@ -18,15 +18,17 @@ export default class Route {
 
   public logger: any;
 
-  public constructor({ req, res, opts }: { req: IM; res: SR; opts: any }) {
+  public constructor({ req, res, opts }: { req: IM; res: SR; opts?: any }) {
     this.req = req;
     this.res = res;
 
-    if (!this.disableRequestLogger && opts.disableRequestLogger) {
-      this.disableRequestLogger = opts.disableRequestLogger;
+    const options = opts || {};
+
+    if (!this.disableRequestLogger && options.disableRequestLogger) {
+      this.disableRequestLogger = options.disableRequestLogger;
     }
-    if (!this.disableErrorHandler && opts.disableErrorHandler) {
-      this.disableErrorHandler = opts.disableErrorHandler;
+    if (!this.disableErrorHandler && options.disableErrorHandler) {
+      this.disableErrorHandler = options.disableErrorHandler;
     }
   }
 

diff --git a/tests/route.ts b/tests/route.ts
@@ -0,0 +1,13 @@
+import { Route } from '../src/index';
+
+describe('route', () => {
+  describe('without opts', () => {
+    it('does not throw', async () => {
+      expect.assertions(1);
+      expect(() => new Route({
+        req: ({} as any),
+        res: ({} as any),
+      })).not.toThrow();
+    });
+  });
+});