Fixes issues in testing

[ksraj123]

Problem

Github Issue Number: #152 , #161

Solution of problem

Issue 152

• Updated routes in test file
• db operation was being carried out after db was already cleared by beforeEach, this caused an mongoose error, added await to async task
• updated tests to expect the right status code, redirection gives 302

Issue 161

• fixed 400 issue in user route, GET /user/invite expected role in query string but the test did not provide any, due to this the decrypted token in the controller of GET /user/invite/:token did not had a role property which caused it to return 400
• fixes error address in use error for all test files, added a guard to app.js to start server only when the environment is not testing, which was causing this issue
• fixed issues with propsal route, Proposal schema did not have a organization property this caused the json returned from the controller of POST /proposal to not have organization property as well which was the reason for the test failing. Added the organization property to Proposal schema but did not made it required to ensure something else does not break due to this.
• fixed jest not exiting, some tests started the server in beforeAll but did not close it in afterAll which was causing this issue.

Type of Change

• [x] Bug fix
• [ ] New Feature
• [ ] Development of UI/UX prototypes
• [x] Small refactor
• [ ] Change in Documentation

Checklist

• [x] My code follows the same style as the codebase
• [ ] My Code change requires a change in documentation
• [ ] I have updated the Readme accordingly
• [x] I made PR against development branch
• [x] I have run the test cases locally and it's passing.
• [ ] I have squashed my commits

[ksraj123]

@Rupeshiya @AuraOfDivinity @devesh-verma Please review

[Rupeshiya]

Hi @ksraj123, Could you please run the test case locally and add the screenshot here?

[ksraj123]

@Rupeshiya Sure, Here it is.

[Rupeshiya]

Cool, Looks fine!!

[AuraOfDivinity]

Looks good for me!

[ksraj123]

• EADDRINUSE Issue fixed for all routes

• Issues in proposal route fixed

• Fixed issues with org route

• All test cases passing

[codecov[bot]]

Codecov Report

Merging #160 into development will decrease coverage by 5.33%. The diff coverage is 55.45%.

@@               Coverage Diff               @@
##           development     #160      +/-   ##
===============================================
- Coverage        68.85%   63.51%   -5.34%     
===============================================
  Files               26       46      +20     
  Lines              687     1472     +785     
  Branches            94      180      +86     
===============================================
+ Hits               473      935     +462     
- Misses             191      477     +286     
- Partials            23       60      +37     

Impacted Files	Coverage Δ
app/controllers/auth.js	83.33% <ø> (-2.39%)	:arrow_down:
app/middleware/auth.js	93.33% <ø> (ø)
app/models/Event.js	61.90% <ø> (ø)
app/models/Organisation.js	55.55% <ø> (ø)
app/models/Post.js	66.66% <ø> (ø)
app/models/Project.js	60.00% <ø> (ø)
app/models/UrlShortner.js	100.00% <ø> (ø)
app/routes/index.js	83.33% <ø> (+8.33%)	:arrow_up:
app/routes/urlShortner.js	100.00% <ø> (ø)
app/utils/response-helper.js	37.50% <0.00%> (ø)
... and 56 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 48e8523...ab05538. Read the comment docs.