CMU-313 / nodebb-f24-boba

cmu-313-f24-nodebb-f24-NodeBB created by GitHub Classroom
GNU General Public License v3.0
0 stars 0 forks source link

Vicky's merges for src/middleware/render.js from Project 1 and tests from tests/middleware.js #14

Closed vickyc2266 closed 5 days ago

vickyc2266 commented 5 days ago

Reduced cognitive complexity of the renderMethod which is within the processRender function of the render.js file by taking away a conditional that appears in the code more than once and putting it into a helper function with an intuitive title called checkHeadersSent(res). The function here is supposed to provide the same exact logic as the original code but just putting it in into a function and calling that function where it is needed.

UI Test Login Successfully as admin Able to Run Server Successfully Able to access different and render different UI pages

coveralls commented 5 days ago

Pull Request Test Coverage Report for Build 10974935758

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/middleware/render.js 5 6 83.33%
<!-- Total: 5 6 83.33% -->
Totals Coverage Status
Change from base Build 10841145095: 0.007%
Covered Lines: 22325
Relevant Lines: 25583

💛 - Coveralls