fix: setTheme now works correctly when a function is passed.

[trm217]

Description

Fixes #277. The dependencies of the set-state function, that is a useCallback hook it's self were faulty. This PR defined 'theme' as a dependency of the callback-hook and removes 'forcedTheme' from the dependency. Additionally a test-case has been added where setState is called with (theme) => theme === 'light' ? 'dark' : 'light')

This PR is based on: #285

Commits

• test: start refactoring tests to use renderHook
• fix: add 'theme' as a dependency to useCallback of 'setTheme' and rename parameter to value
• test: add test for setting theme via a callback while using the previous theme value
• docs: add text to describe accepted parameters of 'setTheme'
• fix: remove 'forcedTheme' as a dependency from the 'setTheme' calback as it's not needed

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
next-themes-basic	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 24, 2024 4:04pm
next-themes-tailwind	❌ Failed (Inspect)			Apr 24, 2024 4:04pm
next-themes-with-app-dir	❌ Failed (Inspect)			Apr 24, 2024 4:04pm