test: separate files for different test cases

[sarayusreeyadavpadala]

Hey there! I am a beginner to Open Source and I am interested to contribute to Open Food Facts. Here, I've separated the test cases into different files and deleted the existing file. Let me know if any changes are needed. Looking forward to your suggestions.

[alexgarel]

@sarayusreeyadavpadala awesome to see your contributions.

I don't have time to review right now but I will maybe next week (if @aadarsh-ram does not have the time in-between).

[sarayusreeyadavpadala]

@alexgarel No problem, take your time.

[sumit-158]

@sarayusreeyadavpadala Initial comments please use "make lint"

[sarayusreeyadavpadala]

@sarayusreeyadavpadala Initial comment please use "make lint"

@sumit-158 Kindly elaborate further on what you have just said.

[aadarsh-ram]

Hi @sarayusreeyadavpadala! Thank you for the contributions. Currently, your PR is failing workflows, related to code quality and linting. Please use the make lint command at the root of the repo and commit again. Someone will review your PR soon.

[sarayusreeyadavpadala]

Hi @sarayusreeyadavpadala! Thank you for the contributions. Currently, your PR is failing workflows, related to code quality and linting. Please use the make lint command at the root of the repo and commit again. Someone will review your PR soon.

Thank you @aadarsh-ram
I shall attend to that task promptly within the day.

[sarayusreeyadavpadala]

@aadarsh-ram, @sumit-158 Could you please check my new commit once....

[sumit-158]

@sarayusreeyadavpadala Seems like the Linting is not fixed yet :-) , make sure to use make lint. Lets us know if you face any issue 🙂 Please refer this link

[sarayusreeyadavpadala]

@sarayusreeyadavpadala Seems like the Linting is not fixed yet :-) , make sure to use make lint. Lets us know if you face any issue 🙂 Please refer this link

Thank you @sumit-158

[sarayusreeyadavpadala]

@sarayusreeyadavpadala Seems like the Linting is not fixed yet :-) , make sure to use make lint. Lets us know if you face any issue 🙂 Please refer this link

@sumit-158 when I run "make lint" command, it shows-
docker-compose run --rm --no-deps facets-api isort .
process_begin: CreateProcess(NULL, docker-compose run --rm --no-deps facets-api isort ., ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [Makefile:20: lint] Error 2

[sumit-158]

@stephanegigandet Can you use Slack for further discussion? Join us in the channel #Knowledge-panel

[sarayusreeyadavpadala]

I have sorted the imports using "isort" this time. And also used "black".

[sarayusreeyadavpadala]

Could anyone approve the workflows.

[sumit-158]

Hi @sarayusreeyadavpadala good work, I have made a push for the lint issue, and now it is resolved but tests are failing, please look into it :-)
tip: use docker-compose run --rm --no-deps facets-api pytest -vv . to see the total output of tests and why and where it is failing

[sarayusreeyadavpadala]

Hi @sarayusreeyadavpadala good work, I have made a push for the lint issue, and now it is resolved but tests are failing, please look into it :-) tip: use docker-compose run --rm --no-deps facets-api pytest -vv . to see the total output of tests and why and where it is failing

Thank you @sumit-158 I'll do that.