search

COP4331-LargeGroupProject-Fall22 / backend

Backend for SmartChef project, see https://github.com/COP4331-LargeGroupProject-Fall22/SmartChef
0 stars 1 forks source link

Add user authentication #15

Closed mplekunov closed 1 year ago

mplekunov commented 1 year ago

Completely implemented server-side User interface + user authentication

mplekunov commented 1 year ago

Two more comments, sorry:

1. All of your pull requests should create corresponding swaggerhub documentation with a working example to show that the API works correctly

2. All of your pull requests should have corresponding [unit and integration](https://blog.logrocket.com/unit-and-integration-testing-for-node-js-apps/) tests

I just noticed that comment... Will be done.

mplekunov commented 1 year ago

Yet to figure out how to do complete API tests on the endpoints.. It's not as easy as it might seem to be.

mplekunov commented 1 year ago

I figured out how to do mocking of database/middleware/API endpoints/classes. The last commit I made has implementation of some tests. They are not "fully" completed, however, I simply got tired of thinking about testing and writing tests so the complete variation of tests will be made sometime later (we could create an issue regarding tests).

At this point I will start doing other stuff, like integration of recipes and other features.. When I'm going to be done with them then I switch back to testing.