Skip to content

Commit

Permalink
add todos
Browse files Browse the repository at this point in the history
  • Loading branch information
Carmine DiMascio committed Mar 25, 2019
1 parent 9aa45bf commit 8914724
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,7 @@
Note: app.params will not have registered for an unknown path param i.e. when express defines a different path name param than express (we should attempt to detect this and flag it)
- (done) add tests with an indepently defined router
- (done) throw error (e.g 404) when route is defined in express but not in openapi spec
- test with merge middleware
- exclude routes
- only throw not found on routes with a openapi base path
- test with merge middleware
- test with case insensitive and case sensitive routing

0 comments on commit 8914724

Please sign in to comment.