Implement TDD for General CMD

[rafiramadhana]

Relates to https://github.com/warnet-gi/WarnetBot/issues/54

This PR is initiating TDD by:

• Add unit test for General cog
• Add workflow for unit test and coverage
• Add .env.test (it looks like our app does not start if we rely on the .env.example)

Some notes:

• This MR only covers General.ping and General.nobar
• I haven't written any helper methods (like in tests/helpers.py) because I think they are no yet needed
• I'm not that familiar with poetry, so I proceed by using pip in the workflow
• In my local, I did pip install pytest-cov and pip freeze >> requirements.txt

PS. This PR is very open for discussions

[Iqrar99]

The test you provided looks promising, It'll be a good starting point to write other tests. I'll try to learn more about utilizing this magic mock

anyway, you can just add the pytest-cov under dev section here along with the requirements.txt I explained above https://github.com/warnet-gi/WarnetBot/blob/01cbd9fdb80b29fd83a495adeafd36bd1869d44a/pyproject.toml#L21-L24

[rafiramadhana]

anyway, you can just add the pytest-cov under dev section here along with the requirements.txt I explained above

got it

[rafiramadhana]

@Iqrar99 btw do you have concerns for the .env.test?

the reason why I don't use .env.example is because the code expects GUILD_ID and DEV_GUILD_ID to be numeric

while they are set as alphabet in the .env.example

GUILD_ID="YOUR_GUILD_ID"
DEV_GUILD_ID="YOUR_DEV_GUILD_ID"

i think, we can also update the .env.example instead of adding .env.test to reduce the number of files within the codebase

---GUILD_ID="YOUR_GUILD_ID"
+++GUILD_ID="123"
---DEV_GUILD_ID="YOUR_DEV_GUILD_ID"
+++DEV_GUILD_ID="123"

wdyt?

PS. in this PR, to avoid breaking in other developer's env (which uses .env.example) i chose to add a new file .env.test instead

[Iqrar99]

i think, we can also update the .env.example instead of adding .env.test to reduce the number of files within the codebase

Have you tried to change the guild id variables in .env.example to numeric? Does it work with the test? if yes, then it sounds good. Then, we can add a comment suggesting people change the env variables to their own guild id.

[rafiramadhana]

Have you tried to change the guild id variables in .env.example to numeric? Does it work with the test?

yes. ok, i will update the .env.example