Test cases!

[ryansurf]

New test cases are always welcome, no matter how simple! This can be a good way to learn how to write simple unit tests and learn PyTest.

Tests are written in /tests

[K-dash]

@ryansurf
Hi👋

I think the first thing we should tackle is separating the test code by file.
As a starting point, I'll try creating the unit tests for Flask on my end:)

[ryansurf]

@K-dash agreed.

I'll separate the current tests in test_code.py into their proper files. I'm thinking files like test_helper.py, test_apis.py, test_server.py to test helper functions, functions that make API calls and the flask server, respectively. Thoughts?

[K-dash]

@ryansurf

I agree :)
I was thinking of splitting the test files exactly as you mentioned.

Also, since pytest can generate test coverage reports, I would like to add that as well.

[ryansurf]

@K-dash Haha nice

I actually didn't even know that was a thing, just googled it. That's super cool. When I made test_code.py that was the first time I've implemented testing, still getting the hang of best practices.

Do you wanna take care of this one? I'm logging off for the night, I can do it tomorrow unless you wanna tackle it :0

[K-dash]

@ryansurf
I'd be happy to 👌
I'll take care of adding the test cases and coverage report for test_server.py.

[K-dash]

@ryansurf
I will start working on this once #21 is merged :)

[K-dash]

@ryansurf
Thanks for the merge!
I'll start implementing the test code.

[taahamultani]

Hello @ryansurf
Is @K-dash working on Implementing Unit test cases? If not, I'd like to contribute.
Additionally, I was thinking of using sonarqube to improve code smells, duplications and more.

[K-dash]

@taaaahahaha Hi!
Since I am not working on this issue right now, please try to work on it!

[taahamultani]

Sure, I'm happy to.

Do assign the issue to me and I'll get started.
@ryansurf

[ryansurf]

@taaaahahaha

Sounds good, let me know if you need any help

[taahamultani]

Hello,
@ryansurf @K-dash
I've went through the library, and here are my thoughts.

• Overall 86% code coverage is already achieved. But, edge cases like failures and different inputs aren't covered. Should this issue focus on increasing the code coverage or implementing those edge cases, or both.
• What should we do about those functions which have openAPI keys; Is there a sample key which can be used to cover those lines?
  Thanks!!