Remove unused 'placement' prop

[mrkeshav-05]

Proposed change

This PR resolves the sonarcloud issue which is placement PropType is defined but prop is never used.

Resolves: #2889

Checklist

• Required: I read and followed the contributing guidelines
• Required: I ran make check-test locally and all tests passed
• I used AI for code, documentation, or tests in this PR

[coderabbitai]

Summary by CodeRabbit

Release Notes

• Tests
  • Updated test mock configurations for improved test maintainability.

---

Note: This release contains no user-facing changes.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Removes the unused placement?: string property from the MockTooltipProps interface in the Card component test file. This is a direct response to issue #2889 identifying an undeclared but unused prop in the test mock type definition.

Changes

Cohort / File(s)	Summary
Test mock type cleanup frontend/__tests__/unit/components/Card.test.tsx	Removed unused optional property placement?: string from MockTooltipProps interface

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• feat: add comprehensive unit tests for Card component #1856 — Modifies the same test file and tooltip mock type usage patterns
• Fix: Ensure tooltips appear on all cards instead of only the first one. #1054 — Related changes to tooltip placement property definition and usage

Suggested labels

frontend-tests

Suggested reviewers

• arkid15r
• kasya

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing the unused 'placement' prop from the MockTooltipProps interface in Card.test.tsx.
Description check	✅ Passed	The description clearly explains the change resolves a SonarCloud issue about an unused 'placement' PropType and links to issue #2889.
Linked Issues check	✅ Passed	The PR successfully implements the requirement from issue #2889 by removing the unused 'placement' prop from MockToolTipProps interface in Card.test.tsx.
Out of Scope Changes check	✅ Passed	All changes are in scope; only the unused 'placement' property was removed from the test file interface as required.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7a08db7 and b8fb6ce.

📒 Files selected for processing (1)

• frontend/__tests__/unit/components/Card.test.tsx (0 hunks)

💤 Files with no reviewable changes (1)

• frontend/tests/unit/components/Card.test.tsx

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud