search

40-percent / 40percent-bot

MIT License
11 stars 10 forks source link

Add unit test framework #10

Open FrancisUsher opened 3 years ago

FrancisUsher commented 3 years ago

This would be a huge value added to this project.

Here's an example tutorial of setting up something similar: https://medium.com/@aussy/lets-create-a-discord-music-bot-using-test-driven-development-tdd-part-1-9c95ea32766f

alferguet commented 3 years ago

I am eager to work on this in order to get more experience with TDD and refresh my Jest knowledge based on previous Nestjs projects.

btmccollum commented 3 years ago

Is this actively being worked on at this point?

alferguet commented 3 years ago

Not by me at least, it was commented in the discord that other user would take the initiative with this one as got little time left. If it is still not assigned can start once more into it.

btmccollum commented 3 years ago

Not by me at least, it was commented in the discord that other user would take the initiative with this one as got little time left. If it is still not assigned can start once more into it.

Yep that was me asking about it, just didn't want to step on anyone's toes if they were pretty deep on this already.

FrancisUsher commented 3 years ago

Is this actively being worked on at this point?

Please have at it!

FrancisUsher commented 3 years ago

I would consider this feature accepted if we had coverage over at least one of the event handlers and its call tree. Just enough to demo how the test technique applies to our discord.js application at all layers

ashsobeck commented 3 years ago

going to take a stab at this in the next few days if no one else is working on it!