search

waiyan93 / web-scraping

0 stars 0 forks source link

Low test coverage #11

Open olivierobert opened 2 years ago

olivierobert commented 2 years ago

Issue

At the beginning of the implementation, several unit tests were added for the core business logic (i.e., keyword listing, keyword details) but then have been removed (see this test deletion). In the end, there are only unit tests for the Laravel-based authentication.

Expected

The model, controllers, jobs, etc. - basically all the classes related to keyword processing - must have unit tests.

waiyan93 commented 2 years ago

Sorry about that sir. Due to the unstable electricity and connection, I skipped the testing to meet with the deadline. I will write them.