chore: Refactor TokenDissociateTransaction protobuf body creation

[manishdait]

Description:
This PR refactors _build_proto_body() in TokenDissociateTransaction to avoid raising a ValueError when account_id or token_ids are not set. This aligns the behavior of the other SDKs and fixes the TCK test

Changes Made:

• Avoid throwing ValueError when account_id or token_ids is None
• Added unit/e2e test to validate changes

Fixes #2453

Notes for reviewer:
Expected behavior (to match with the other SDKs):

• If account_id is provided and token_ids is skip, the transaction should execute successfully.
• If account_id is skip, the transaction should fail with a PrecheckError rather than raising ValueError.

Checklist

• Documented (Code comments, README, etc.)
• Tested (unit, integration, etc.)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main    #2454      +/-   ##
==========================================
+ Coverage   95.00%   95.01%   +0.01%     
==========================================
  Files         164      164              
  Lines       10461    10464       +3     
==========================================
+ Hits         9938     9942       +4     
+ Misses        523      522       -1     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Walkthrough

Token dissociation protobuf construction now conditionally includes account and token fields, filters null token identifiers, and supports omitted inputs. Unit and integration tests cover field serialization, successful execution, and missing-account precheck failure.

Changes

Token dissociation handling

Layer / File(s)	Summary
Conditional protobuf body construction src/hiero_sdk_python/tokens/token_dissociate_transaction.py, tests/unit/token_dissociate_transaction_test.py	_build_proto_body conditionally populates fields, ignores null token identifiers, and is covered for present, missing, and empty inputs.
Dissociation execution scenarios tests/integration/token_dissociate_transaction_e2e_test.py	Integration tests use the env fixture to verify standard dissociation, execution without token IDs, and PrecheckError when the account ID is missing.

Estimated code review effort: 3 (Moderate) | ~20 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: refactoring TokenDissociateTransaction protobuf body creation.
Description check	✅ Passed	The description is directly related to the code changes and test coverage added in the PR.
Linked Issues check	✅ Passed	The changes address #2453 by allowing dissociation with no token IDs and adding coverage for that case.
Out of Scope Changes check	✅ Passed	The PR stays focused on TokenDissociateTransaction behavior and related tests, with no clear unrelated changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[coderabbitai]

Actionable comments posted: 2

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: e603a48d-a85d-4960-99c5-856acd31bfb9

📥 Commits

Reviewing files that changed from the base of the PR and between 832a5e9 and bbdea50.

📒 Files selected for processing (3)

• src/hiero_sdk_python/tokens/token_dissociate_transaction.py
• tests/integration/token_dissociate_transaction_e2e_test.py
• tests/unit/token_dissociate_transaction_test.py