[Good First Issue]: Rename `OperatorEqual*` test cases in `TokenNftTransferUnitTests` to follow `OperatorEquals` naming convention

[rwalworth]

🐥 Good First Issue

This issue is a great fit for first-time contributors looking to get familiar with the Hiero C++ SDK codebase.

Good First Issues are intentionally small and well-scoped. The change is a straightforward rename with no logic to implement — you just need to find the right lines and update the test names to match the established pattern.

---

👾 Description of the Task

The equality tests added to TokenNftTransferUnitTests in PR #1371 use the prefix OperatorEqualWith, but the established naming convention across the SDK uses OperatorEquals (with a trailing s).

For reference, see these existing files:

• ExchangeRateUnitTests.cc uses OperatorEqualsDefaults, OperatorEqualsSimilar, OperatorEqualsDiff
• TokenTransferUnitTests.cc uses OperatorEqualsSame, OperatorEqualsDiff

The six test cases added in PR #1371 need to be renamed to match this convention.

Relevant file:

src/sdk/tests/unit/TokenNftTransferUnitTests.cc

---

💡 Proposed Approach

Rename the six TEST_F macros in TokenNftTransferUnitTests.cc. No logic changes — only the test name strings change.

---

👩‍💻 Implementation Steps

• Open src/sdk/tests/unit/TokenNftTransferUnitTests.cc

• Apply the following exact renames:

  Current name	New name
  OperatorEqualWithDefaultConstructedInstances	OperatorEqualsDefaultConstructed
  OperatorEqualWithIdenticalConstructedInstances	OperatorEqualsSame
  OperatorEqualWithDifferentNftId	OperatorEqualsDiffNftId
  OperatorEqualWithDifferentSenderAccountId	OperatorEqualsDiffSenderAccountId
  OperatorEqualWithDifferentReceiverAccountId	OperatorEqualsDiffReceiverAccountId
  OperatorEqualWithDifferentIsApproval	OperatorEqualsDiffIsApproval

• Build and run the unit tests to confirm all six renamed tests still pass:

  ctest -j6 -C Debug --test-dir build/linux-x64-debug -R TokenNftTransfer

• Open a pull request with the change

---

✔️ Acceptance Criteria

• All six test cases in TokenNftTransferUnitTests.cc are renamed exactly as shown in the table above
• No other changes are made to the file
• All renamed tests pass

---

📋 Step-by-Step Contribution Guide

• Comment /assign to request the issue
• Wait for assignment
• Fork the repository and create a branch
• Set up the project using the instructions in README.md
• Make the requested changes
• Sign each commit using -s -S
• Push your branch and open a pull request

Read Workflow Guide for step-by-step workflow guidance. Read README.md for setup instructions.

❗ Pull requests cannot be merged without S and s signed commits. See the Signing Guide.

---

🤔 Additional Information

• Originated from nitpick left during review of PR feat: add TokenNftTransfer equality operator #1371
• Reference naming examples: ExchangeRateUnitTests.cc, TokenTransferUnitTests.cc
• No API or behavior changes — test names only

If you have questions while working on this issue, feel free to ask! Hiero-SDK-C++ Discord

[HarK-github]

/assign

[github-actions]

👋 Hi @HarK-github, welcome to the Hiero C++ SDK community! Thank you for choosing to contribute — we're thrilled to have you here! 🎉

You've been assigned this Good First Issue, and the Good First Issue Support Team (@hiero-ledger/hiero-sdk-good-first-issue-support) is ready to help you succeed.

The issue description above has everything you need: implementation steps, contribution workflow, and links to guides. If anything is unclear, just ask — we're happy to help.

If you realize you cannot complete this issue, simply comment /unassign to return it to the community pool.

Good luck, and welcome aboard! 🚀