amoralyn / dms-api Goto Github PK

your create user, documents and role can be broken down into functions

The user.controller.js is over 200 lines of code.
Refactoring this class would make it simple and easy to read.

Export format for app in the express.js file is inconsistent with the rest of the codebase<you exported the app as {app} where in other files you exported as just app>

Test case for Documents Endpoint:

    -- is poorly implemented. The case should test the length of the returned collection is equal to the limit passed in not a limited doc as you titled that test case(@ Line 190)
    -- has redundant lines 193 and 195
    -- has an error in line 196. It's supposed to test the returned collection length against the set limit, not 0

redundant use of error and success response. It can be converted into a function and reused

No badge showing report of test coverage