test: add tests for integrate onboarding form

[MayankBansal12]

Date: 31-01-26

Developer Name: @MayankBansal12

---

Issue Ticket Number:-

• feat: integrate APIs in onboarding form for create application #1140

Description:

• Add tests for new stepper (create application)
• Add tests for status card (tracka application)

Is Under Feature Flag

• Yes
• No

Database changes

• Yes
• No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

• Yes
• No

Is Development Tested?

• Yes
• No

Tested in staging?

• Yes
• No

Add relevant Screenshot below ( e.g test coverage etc. )

tests

[coderabbitai]

Walkthrough

This PR introduces safer JSON parsing for step data retrieval by adding a safeParse utility function with error handling. The component is refactored to use this utility, and comprehensive integration tests are added for NewStepper and StatusCard components covering submission flows, error handling, and localStorage cleanup.

Changes

Cohort / File(s)	Summary
Storage Utility Enhancement app/utils/storage.js	Added new safeParse(key) function that wraps JSON.parse in try-catch, returning an empty object on parse failure for safer localStorage value retrieval.
Component Refactoring app/components/new-join-steps/new-step-six.js	Replaced five inline JSON.parse calls with safer safeParse() utility in loadAllStepData, improving error resilience without changing control flow.
NewStepper Integration Tests tests/integration/components/new-stepper-test.js	Expanded test coverage significantly with mocked services (router, toast), step data fixtures, and new test scenarios including submission flow (step 6), error handling (409 conflict, API errors, network failures), localStorage cleanup verification, and data aggregation assertions.
StatusCard Integration Tests tests/integration/components/status-card-test.js	Enhanced tests with helper functions, router/onboarding service stubs, and new test module for track application button behavior including navigation verification and missing data handling.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A safer parse, no crashes in sight,
JSON wrapped tight in a try-catch delight,
Tests multiply with each step we take,
From storage to router, no errors we make!
The stepper now stronger, the step-six so bright! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'test: add tests for integrate onboarding form' accurately reflects the main objective of the pull request: adding tests for the onboarding form components (new stepper and status card).
Description check	✅ Passed	The PR description directly aligns with the changeset—it documents adding tests for the new stepper (create application) and status card (track application), which matches the test file modifications and utility refactoring included.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch test/integrate-onboarding-form

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[MayankBansal12]

@coderabbitai review bro

[coderabbitai]

@MayankBansal12: I'll review the changes in this PR for you! 🔍

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@tests/integration/components/new-stepper-test.js`:
- Around line 9-12: The forEach callbacks in this test file (notably in the
clearAllStepData function that iterates STEP_DATA_STORAGE_KEY and the other
forEach usages in the same file) use expression-bodied arrow callbacks which
implicitly return values and trigger Biome errors; change those callbacks to use
block bodies instead (e.g., replace single-expression arrow callbacks with
braces and a statement body so they do not return a value), updating
clearAllStepData and the other forEach occurrences to use block-bodied callbacks
that call localStorage.removeItem(...) within { }.