fix: busy/not-answered calls stay in agent worklist after call closure

[vishwab1]

📋 Description

JIRA ID: AMM-2315

Please provide a summary of the change and the motivation behind it. Include relevant context and details.

Summary

• When an agent closes a call with reason "Number busy", "No reply",
  "Switched off" etc., the record was disappearing from their worklist.
• Root cause: a previous change unallocated ALL disconnected calls
  unconditionally, including busy/not-answered ones.
• Fix: unallocation is now skipped when reasonForCallNotAnswered is
  a not-answered reason (busy, switched off, no reply, etc.).

Changes

• CallClosureImpl.java: Added condition in disconnected block —
  unallocation only happens when reason is NOT a busy/not-answered type.
• Removed redundant re-allocation lines from the busy block
  (no longer needed after fixing root cause).

Test Plan

• Close call with "Number busy" → record stays in worklist
• Close call with "Switched off" → record stays in worklist
• Disconnect call (genuine disconnect, no reason) → record goes to unallocated pool
• Max attempts reached → record removed from worklist as before

---

✅ Type of Change

• 🐞 Bug fix (non-breaking change which resolves an issue)
• ✨ New feature (non-breaking change which adds functionality)
• 🔥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
• 🛠 Refactor (change that is neither a fix nor a new feature)
• ⚙️ Config change (configuration file or build script updates)
• 📚 Documentation (updates to docs or readme)
• 🧪 Tests (adding new or updating existing tests)
• 🎨 UI/UX (changes that affect the user interface)
• 🚀 Performance (improves performance)
• 🧹 Chore (miscellaneous changes that don't modify src or test files)

---

ℹ️ Additional Information

Please describe how the changes were tested, and include any relevant screenshots, logs, or other information that provides additional context.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 601aeaa0-96da-40da-bcdc-30ed81c1d05e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch vb/ecd-call-fix

---

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