Fix signer_nonce serialization in ContractFunctionResult

[utkarsh232005]

This PR fixes a serialization bug in ContractFunctionResult._to_proto() and adds corresponding unit test coverage. Fixes #2251

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f6cf3dc2-e568-42dc-b4e5-e65d43780f30

📥 Commits

Reviewing files that changed from the base of the PR and between 679e4a6 and d97a747.

📒 Files selected for processing (2)

• src/hiero_sdk_python/contract/contract_function_result.py
• tests/unit/contract_function_result_test.py

---

Walkthrough

ContractFunctionResult._to_proto() now conditionally wraps signer_nonce in Int64Value only when not None, preventing silent conversion of absent nonce to zero. A unit test validates both absent and zero cases.

Changes

signer_nonce Optional Field Serialization

Layer / File(s)	Summary
Conditional signer_nonce Int64Value wrapping src/hiero_sdk_python/contract/contract_function_result.py, tests/unit/contract_function_result_test.py	_to_proto() now conditionally wraps signer_nonce in Int64Value only when self.signer_nonce is not None, preventing the protobuf field from silently defaulting to zero when absent. Test verifies field presence via HasField() when None and confirms encoded value when set to 0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Fix signer_nonce serialization in ContractFunctionResult' clearly and specifically summarizes the main change—fixing a serialization bug in the signer_nonce handling.
Description check	✅ Passed	The description accurately relates to the changeset, explaining the serialization bug fix and referencing the linked issue #2251.
Linked Issues check	✅ Passed	The PR fully addresses all coding requirements from issue #2251: conditionally wraps signer_nonce in Int64Value only when not None, adds unit test coverage for the None case, and preserves the distinction between signer_nonce=0 and omitted field.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to fixing the signer_nonce serialization bug identified in issue #2251—no extraneous modifications are present.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

📋 Issue Planner

Built with CodeRabbit's Coding Plans for faster development and fewer bugs.

View plan used: #2251

✨ 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 and usage tips.}

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff           @@
##             main    #2293   +/-   ##
=======================================
  Coverage   93.97%   93.97%           
=======================================
  Files         163      163           
  Lines       10408    10408           
=======================================
  Hits         9781     9781           
  Misses        627      627           

🚀 New features to boost your workflow:

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

[Dosik13]

lgtm

[manishdait]

@utkarsh232005, Please update the branch

[AntonioCeppellini]

LGTM 🛩️ :D
thank you for your contribution!

[github-actions]

👋 Hi @utkarsh232005! Great work completing a Beginner issue! 🎉

Thanks for your contribution! 🚀

Here are some issues you might want to explore next:

• [Intermediate]: Normalize Mnemonic Passphrases to NFKD
• [Intermediate]: Integrate `include-what-you-use` into CMake Build System
• [Intermediate]: Add HIP-869 Dynamic Address Book Example
• [Intermediate]: Add HIP-632 Hedera Account Service (HAS) System Contract Example
• [Intermediate]: Support Jumbo EthereumTransaction (HIP-1086)

🌟 Stay connected:

• ⭐ Star this repository
• 👀 Watch for new issues
• 💬 Join us on Discord

Happy coding! 🚀
— Hiero Python SDK Team