-
Notifications
You must be signed in to change notification settings - Fork 26
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
add unit test for useChatActions hook #69
add unit test for useChatActions hook #69
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #69 +/- ##
==========================================
+ Coverage 74.29% 82.12% +7.83%
==========================================
Files 43 43
Lines 1023 1024 +1
Branches 245 238 -7
==========================================
+ Hits 760 841 +81
+ Misses 261 182 -79
+ Partials 2 1 -1 ☔ View full report in Codecov by Sentry. |
); | ||
}); | ||
|
||
it('should handle regenerate error', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we add test case for regenerate abort?
Signed-off-by: Yulong Ruan <[email protected]>
Signed-off-by: Yulong Ruan <[email protected]>
Signed-off-by: Yulong Ruan <[email protected]>
027a7d9
to
a5eac23
Compare
Signed-off-by: Yulong Ruan <[email protected]>
Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.