search

GravityWolfNotAmused / PlayerCountDiscordBot

A simple discord bot that updates their status' to show player counts.
MIT License
36 stars 16 forks source link

Test Cases and Docker fix #45

Closed GravityWolfNotAmused closed 2 years ago

GravityWolfNotAmused commented 2 years ago

After the last update, I left an issue with the parsing of docker configuration variables for Application variables. This fixes the possible null, the option of having steam or battle metrics, or none.

I have built test cases around this parsing logic, so that in the future if I break something I know before releasing it. Please note that this issue doesn't extend to the standard configuration as JSON holds a specific format for dictionaries that would cause the parser to fail.

I also moved the logic for setting the channel name to an extension method.

Also!

I have moved all of the documentation to the wiki. It's easier that way ;)

GravityWolfNotAmused commented 2 years ago

@Specker We now have test cases around docker configuration parsing. :)