You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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 useSettingsInternal hook.
Deliverable(s):
New unit test cases that cover the testing of the useSettingsInternal hook.
I could not find the useSettingsInternal hook in the project. From what I understand, the useSettings hook directly accesses the settings and setSettings context and returns them.
I could not find the useSettingsInternal hook in the project. From what I understand, the useSettings hook directly accesses the settings and setSettings context and returns them.
Hey @anandxs! Thank you for pointing it out and apologies for the confusion. You're right to observe that settings hook itself doesn't currently hold any internal logic. I'm contemplating adding a utility function to make updating of settings easier instead of having users directly deal with the setter. Let me ponder over this for a bit more - in the meantime, please feel free to look at other issues if you are keen 🥹
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
useSettingsInternal
hook.Deliverable(s):
New unit test cases that cover the testing of the
useSettingsInternal
hook.Additional Context:
The text was updated successfully, but these errors were encountered: