cleanup types

src/framework/types.ts

@@ -2,6 +2,7 @@
 // import { IOpenAPIRequestCoercer } from 'openapi-request-coercer';
 // import { IOpenAPIRequestValidator } from 'openapi-request-validator';
 // import { IOpenAPIResponseValidator } from 'openapi-response-validator';
+import { Request } from 'express';
 import {
   // IOpenAPISecurityHandler,
   SecurityHandlers,
@@ -135,3 +136,7 @@ export interface OpenAPIFrameworkVisitor {
   visitPath?(context: OpenAPIFrameworkPathContext): void;
   // visitOperation?(context: OpenAPIFrameworkOperationContext): void;
 }
+
+export interface OpenApiRequest extends Request {
+  openapi;
+}

src/index.ts

@@ -1,9 +1,10 @@
 import * as _ from 'lodash';
-import { ExpressApp } from 'express';
+import { Application, Response, NextFunction } from 'express';
 import { OpenAPIFrameworkArgs } from './framework';
 import { OpenApiContext } from './openapi.context';
 import * as middlewares from './middlewares';
 import ono from 'ono';
+import { OpenApiRequest } from './framework/types';
 
 const loggingKey = 'express-openapi-validator';
 
@@ -24,7 +25,7 @@ export function OpenApiValidator(options: OpenApiValidatorOpts) {
   this.context = openApiContext;
 }
 
-OpenApiValidator.prototype.install = function(app: ExpressApp) {
+OpenApiValidator.prototype.install = function(app: Application) {
   const pathParams = [];
   for (const route of this.context.routes) {
     if (route.pathParams.length > 0) {
@@ -34,7 +35,7 @@ OpenApiValidator.prototype.install = function(app: ExpressApp) {
 
   // install param on routes with paths
   for (const p of _.uniq(pathParams)) {
-    app.param(p, (req, res, next, value, name) => {
+    app.param(p, (req: OpenApiRequest, res, next, value, name) => {
       if (req.openapi.pathParams) {
         // override path params
         req.params[name] = req.openapi.pathParams[name] || req.params[name];

test/app.common.ts

@@ -17,22 +17,22 @@ export function routes(app) {
     .Router()
     .post('/', function(req, res, next) {
       res.json({
-        name: `${req.metnod}: /router_1`,
+        name: `${req.method}: /router_1`,
       });
     })
     .get('/', function(req, res, next) {
       res.json({
-        name: `${req.metnod}: /router_1`,
+        name: `${req.method}: /router_1`,
       });
     })
     .get('/:id', function(req, res, next) {
       res.json({
-        name: `${req.metnod}: /router_1/${req.params.id}`,
+        name: `${req.method}: /router_1/${req.params.id}`,
       });
     })
     .get('/:id/best/:bid', function(req, res, next) {
       res.json({
-        name: `${req.metnod}: /router_1/${req.params.id}/best/${
+        name: `${req.method}: /router_1/${req.params.id}/best/${
           req.params.bid
         }`,
       });

test/headers.spec.ts

@@ -3,11 +3,11 @@ import * as request from 'supertest';
 import app from './app';
 
 const packageJson = require('../package.json');
-const basePath = app.basePath;
+const basePath = (<any>app).basePath;
 
 describe(packageJson.name, () => {
   after(() => {
-    app.server.close();
+    (<any>app).server.close();
   });
 
   it('should throw 400 if required header is missing', async () =>

test/routes.spec.ts

@@ -3,11 +3,11 @@ import * as request from 'supertest';
 import app from './app';
 
 const packageJson = require('../package.json');
-const basePath = app.basePath;
+const basePath = (<any>app).basePath;
 
 describe(packageJson.name, () => {
   after(() => {
-    app.server.close();
+    (<any>app).server.close();
   });
   it(`should test something`, () => {
     expect('a').to.equal('a');