Is the error reset during an AbortError redundant?
Opened this issue · 0 comments
BenMcH commented
use-x/src/use-fetch/use-fetch.solution.ts
Lines 110 to 113 in ca70f15
Thanks for putting this test suite together! After completing the useFetch
hook, I went back and compared my solution to the committed solution and noticed that this was the one functionality deviation between what I wrote and the expected solution. I would expect that this setError(null)
is not actually needed, as the error is set to null at the top of the hook on line 91.
I may be missing an edge case, but the test suite didn't appear to be affected by the absence of that line.