add unit test for alertSlice reducer#707
Conversation
|
Thank you @, for creating the PR and contributing to our UltimateHealth project 💗. |
SB2318
left a comment
There was a problem hiding this comment.
Hey! 👋 Thanks for taking the time to put this contribution together.
Before we can get this merged, there are two procedural things we need to address to keep our codebase maintainable:
1. Link the Issue
Could you please update the PR description to mention the specific issue this resolves? (e.g., adding Fixes #123 or Resolves #123 to the description). This automatically links the context and closes the issue when merged, which helps us track the project history.
2. Add Unit Tests for UserSlice
As a standard practice, any new Redux slices or modifications to global state management require test coverage before merging. Please add the corresponding unit tests for the UserSlice. Specifically, ensure we are testing:
- The initial state.
- State transitions for any newly added reducers/actions.
- Edge cases or error states (if you are handling asynchronous thunks).
Let me know if you need any guidance on setting up the tests for the slice, otherwise ping me once you've pushed the updates and I'll gladly take another look! 🚀
SB2318
left a comment
There was a problem hiding this comment.
Please read the changes and cover UserSlice test.
|
Added UserSlice tests and resolved merge conflicts. |
Summary
Added unit test for alertSlice reducer initial state.
Type of Change
Work Area
Testing