brainboxdotcc / DPP

C++ Discord API Bot Library - D++ is Lightweight and scalable for small and huge bots!
https://dpp.dev/
Apache License 2.0
1.07k stars 163 forks source link

fix(test): fix ban unit tests to be dependent upon each other #1200

Closed braindigitalis closed 3 months ago

braindigitalis commented 3 months ago

Fixes the ban set/unset/get unit tests to be properly dependent upon each other's callbacks (callback hell, yay) and also to use the IDs of valid users which should never be on a unit test server:

These 3 will remain valid for the forseeable future. using deleted user ids for bans is not a good idea, they eventually get purged and the side effects of adding bans for them is not doucmented behaviour.

Code change checklist

netlify[bot] commented 3 months ago

Deploy Preview for dpp-dev ready!

Name Link
Latest commit 3f2b333be1052807171c6536176f0c5f514d19e1
Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/6696e281f9372f00082e9efa
Deploy Preview https://deploy-preview-1200--dpp-dev.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.