Refactor bestaande code zodanig dat elk command een aparte functie gebruikt die iets returnt voordat het terug wordt gestuurd.
Gebruik unittests om te kijken of die functie nog wel het juiste returnt.
In theorie eenvoudig, in de praktijk toch lastiger. Uiteindelijk moet er ook een hoop gebeuren op het gebied van de bot zelf, en de onderliggende discordpy classes. Ook moet duplicate code zo veel mogelijk worden ontweken, dus is het eigenlijk ideaal als we het testen ook echt met een bot doen (Stroopwafel Test) bijvoorbeeld.
Plan van aanpak:
1. Refactor bestaande code zodanig dat elk command een aparte functie gebruikt die *iets* returnt voordat het terug wordt gestuurd.
2. Gebruik unittests om te kijken of die functie nog wel het juiste returnt.
In theorie eenvoudig, in de praktijk toch lastiger. Uiteindelijk moet er ook een hoop gebeuren op het gebied van de bot zelf, en de onderliggende discordpy classes. Ook moet duplicate code zo veel mogelijk worden ontweken, dus is het eigenlijk ideaal als we het testen ook echt met een bot doen (Stroopwafel Test) bijvoorbeeld.
Deze PR houdt de voortgang bij van de branch.
Fixes #18
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Plan van aanpak:
In theorie eenvoudig, in de praktijk toch lastiger. Uiteindelijk moet er ook een hoop gebeuren op het gebied van de bot zelf, en de onderliggende discordpy classes. Ook moet duplicate code zo veel mogelijk worden ontweken, dus is het eigenlijk ideaal als we het testen ook echt met een bot doen (Stroopwafel Test) bijvoorbeeld.
Deze PR houdt de voortgang bij van de branch.
Fixes #18
branch is verpest omdat ik iets doms heb gedaan met git, komt een vervangend PR voor
Pull request closed