test(useRecentSearches): verify hook is SSR-safe — loadFromStorage handles undefined window

[JhaSourav07]

🧩 Background

loadFromStorage in hooks/useRecentSearches.ts guards with typeof window === 'undefined'. This path has no test since jsdom always provides window.

🎯 Objective

Add a test that temporarily removes window to verify the SSR guard.

📁 Files to touch

• hooks/useRecentSearches.test.ts

🛠️ Implementation steps

• Before rendering the hook, temporarily delete global.window.
• Assert the hook initializes with an empty searches array.
• Restore global.window after the test.

✅ Definition of done

• 1 test case.
• All existing hook tests pass.

[mahek888]

/claim

[github-actions]

✅ Successfully assigned issue to @mahek888

💡 Please read CONTRIBUTING.md if you haven't already. Good luck! 🚀

[github-actions]

⚠️ Assignment automatically removed due to inactivity.
Feel free to reclaim the issue if you want to continue working on it.

[AnshuS13]

/claim

[github-actions]

🎉 Assigned! Welcome to the project, @AnshuS13.

⏳ Reminder: You have 2 days to submit a Pull Request. After 2 days of inactivity, you will be automatically unassigned to give others a chance (as per our GSSoC anti-hoarding policy).

💡 Please read CONTRIBUTING.md if you haven't already.

Happy coding! 🚀

[AnshuS13]

Hi @JhaSourav07

While working on this issue I noticed that the current implementation of
hooks/useRecentSearches.ts no longer contains the
typeof window === 'undefined' guard mentioned in the issue description.

The current loadFromStorage() implementation relies on a try/catch around
localStorage access instead.

Attempting to delete global.window before renderHook() causes React Testing
Library itself to throw ReferenceError: window is not defined before the hook
runs.

Could you clarify the intended test approach for the current implementation?

[JayalaxmiRN]

/claim

[github-actions]

❌ This issue is already assigned to @AnshuS13