[NoelStephensUnity]

Purpose of this PR

The Pose cases of RpcTypeSerializationTests passed without running. Their branches called the coroutine helpers without yield return, so the RPCs were never sent or checked, and the NativeArray/NativeList cases never disposed their collections. Those collections are the 6 allocations left after #4219 in the unified suite's leak report.

PR Scope:

• Adds the missing yield return to all four Pose branches: single value, managed array, NativeArray and NativeList.

Jira ticket

None.

Documentation

• No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

• Manual testing done

Automated tests:

• Covered by existing automated tests
• Covered by new automated tests

RpcTypeSerializationTests: Server 132/132 and UnifiedServer 132/132 passed, with all Pose cases now running.

Does the change require QA team to:

• Review automated tests?
• Execute manual tests?
• Provide feedback about the PR?

Up-port

Not needed: targets develop-3.x.x. The same Pose branches exist on develop-2.0.0 if a port is wanted.

Backports

None.

[u-pr]

💡 Harness Review

The four added yields correctly execute the existing Pose RPC test coroutines. I examined all changed branches, matching RPC methods, assertions, native collection disposal, and compilation guards; no actionable issues were found.

_{Reviewed commit a1a4505}

_{🤖 Helpful? 👍/👎}

[u-pr]

💡 Harness Review

The four additions correctly run the previously skipped Pose coroutine tests. I examined the matching RPC methods, helper assertions, native collection disposal, and neighboring branches, and found no actionable issues.

_{Reviewed commit 12d8fe3}

_{🤖 Helpful? 👍/👎}

[codecov-github-com]

Codecov Report

All modified and coverable lines are covered by tests ✅

@@                Coverage Diff                @@
##           develop-3.x.x    #4220      +/-   ##
=================================================
- Coverage          76.75%   76.74%   -0.01%     
=================================================
  Files                159      159              
  Lines              28078    28078              
=================================================
- Hits               21550    21549       -1     
- Misses              6528     6529       +1     

Flag	Coverage Δ
NGOv2_project_testproject_ubuntu	75.78% <ø> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

Components	Coverage Δ
com.unity.netcode.gameobjects	76.74% <ø> (-0.01%)	⬇️

ℹ️ Need help interpreting these results?

[u-pr]

💡 Harness Review

The four added yields correctly execute the Pose RPC serialization checks and reach the native collection disposal paths. I examined the coroutine helpers, Pose RPC methods, and neighboring cases and found no actionable issues.

_{Reviewed commit 4e3dd0d}

_{🤖 Helpful? 👍/👎}