-
-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Task] Add unit test cases for useToastsInternal hook #130
Comments
I want to give it a try to this one. |
Go for it! |
Hey @manoshatzi! Checking in to see if you're facing any problems working on the issue. Feel free to reach out! |
Hey @manoshatzi! With hacktoberfest drawing closer to the end, I'll be re-opening this issue for others that may be keen in this final stretch. That said, if you're still interested, feel free to open a PR soonest! |
Hey @tjtanjin. Yes do that. So sorry for not have been finish it but I was really too busy and my spare time was really too less. |
can you please assign this me? I tried it already. @tjtanjin |
@tjtanjin can you assign this to me |
Can you please assign this on the basis of fcfs as I have only one PR left for hackotberfest'24 :)... |
Hey @anjumnnit, I'll assign this issue to you since you requested for it first. I'll keep the PR from @Nandinibajaj16 open in the meantime, in case the issue is left incomplete. Separately @Nandinibajaj16, I've tagged the PR from you with |
@tjtanjin My bad didn't see the updates on the issue before raising the PR, it's okay :) |
tysm 😿 |
Task Description:
Currently, we run unit tests using jest which can be found here.
However, the test cases are not yet comprehensive enough and work can be done to cover more grounds. Specifically for this issue, we hope to add test cases for the
useToastsInternal
hook.Deliverable(s):
New unit test cases that cover the testing of the
useToastsInternal
hook.Additional Context:
The text was updated successfully, but these errors were encountered: