manojpanchal90 / nimbletest

0 stars 0 forks source link

[Chore] Implement test for the services #7

Open olivierobert opened 11 months ago

olivierobert commented 11 months ago

Issue

Automated tests do not cover the core business logic (CSV upload, scraping).

Expected

While 100% test coverage is not required for this code challenge, all critical paths of the application should be unit-tested (and, ideally, UI-tested). So the following services should be covered:

[!NOTE] There is a bit of a challenge to implement tests for a service that generates HTTP requests. Hence, it is an expected part of the submission for us to assess your skills in writing challenging tests.

manojpanchal90 commented 11 months ago

@olivierobert Push initial Spec to the PR. I am still working on remaining specs. Here is the latest PR https://github.com/manojpanchal90/nimbletest/pull/8