chore: enable unified hybrid tests by NoelStephensUnity · Pull Request #4122 · Unity-Technologies/com.unity.netcode.gameobjects · GitHub

Purpose of this PR

Stands up a dedicated CI job that runs the unified (NGO + N4E) hybrid-prefab integration tests, and
fixes the three bugs that were preventing UnifiedNetworkTransformTest from passing.

This is the first part of 3 PRs:

(This PR) gets the initial scaffolding into place.
chore: unified hybrid tests enabled environment var #4137 removes the test filter and uses the UNIFIED_TEST environment variable (almost exactly like CMB service tests with the difference that it is going to start out as a "turned off by default" for all tests and will be an "opt-in". The UniversalNetworkTransformTest is the only opt-in for 4137 ). 4137 also removes a bunch of the YAML comment verbosity along with other script file comments.
Yet to be posted PR that will include more NetworkTransformTests.

Additional unified netcode hybrid testing details

The pin for unified testing is 6000.7.0a5, not 6000.7.0a2 due to conflicts between the two.

The original plan was to pin a2 (breakpoints are more reliable there). a2 turns out to be unusable —
it cannot compile the unified code at all. Both alphas bundle com.unity.netcode 6.7.0, so the
package version does not distinguish them; only the trunk snapshot does. Verified by reading the
sources bundled in each editor:

symbol	a2	a5
IOutOfBandRpcCommand	0	2
OutgoingOutOfBandRpcDataStreamBuffer	0	20

UnifiedNetcodeTransport is gated on #if UNIFIED_NETCODE && OUT_OF_BAND_RPC and needs both.
Separately, a2 still names the component GhostAdapter; a5 renamed it to GhostObject, which is what
the test helpers call. Two independent blockers, so a2 is off the table regardless of debugger quality.

Runtime fixes

UnifiedNetcodeUpdateSystem.OnCreate used the ISystem signature OnCreate(ref SystemState) on a
SystemBase. Without override that is a new method Entities never calls, so neither
RequireForUpdate took effect and OnUpdate ran from the first world tick — before
StartClient/StartServer assign Transport and NetworkManager. CreateSingleWorldHost calls
AppendWorldToCurrentPlayerLoop, so the world is live immediately and any tick in that window was a
NullReferenceException. Corrected to the parameterless override, plus a null guard in OnUpdate.
NetcodeIntegrationTest.SpawnObject set Netcode.Instance.m_ActiveWorld after
Object.Instantiate. The hybrid prefab is active, so the clone's GhostObject.Awake runs
synchronously inside Instantiate; the clone is not a prefab, so Awake acquires an entity reference
and resolves the target world from that singleton. N4E's rate managers reassign it on every world
update, so it pointed at whichever client world updated last and the spawn was rejected with
"You can only spawn a ghost on a server or during prediction on a client." Assignment hoisted above
Instantiate.
NetworkObject.InitGhost threw when a hybrid prefab was created before any NetworkManager exists,
which is the normal ordering in integration tests. Now returns early on a null NetworkManager.

CreateHybridPrefab also now calls GhostObject.InitializeAsPrefab() instead of hand-rolling the
GhostPrefabReference setup, which picks up N4E's try/finally reset of s_IsPostProcessing.
UnifiedNetworkTransformTest was hardened: position is validated on initial spawn as well as after
the move, and the fixed WaitForSeconds(1) is replaced with WaitForConditionOrTimeOut reporting
which client diverged and by how much.

Tradeoffs worth reviewer attention

Duplicate manifest. manifest-unified.json is a full copy of manifest.json. It differs by more
than the N4E entry: manifest.json pins the builtin versions that exist on 6000.6
(addressables 2.11.1, timeline 6.6.0, ugui 2.6.0), which do not resolve on 6000.7.0a5. That
means hand-syncing two manifests until N4E becomes a hard NGO dependency, at which point this goes
away. I could not find a way to avoid the duplication without dropping 6000.6 support.
Test selection is a name pattern. unified_test_filter: "*Unified*" matches the NUnit full test
name, which covers the dedicated fixtures and is intended to also cover shared fixtures
parameterized with HostOrServer.UnifiedHost/UnifiedServer. An NUnit [Category] would be less
fragile once more fixtures gain unified variants; there is no [Category] usage in the package today
and UnifiedTestRunner's --category support is unconfirmed, so this is deliberately deferred.
This is in the PR gate. run_all_unified_tests is a dependency of pr_code_changes_checks,
wired the same way as the CMB service tests. That puts an unsupported alpha editor into the mandatory
gate: when N4E lands breaking changes in trunk this goes red and the pin has to be bumped before any
PR can go green. Chosen deliberately over an opt-in trigger so unified breakage is noticed
immediately. unified_pr_checks (/ci unified) remains for PRs the main gate does not cover.
Bumping the pin means changing three things together: unified_editors.default,
unified_netcode_version, and the com.unity.netcode version in manifest-unified.json.

Known gap: the CI job runs editor playmode; verification so far is a standalone player run.
See Testing & QA below.

Jira ticket

MTT-XXXX

Documentation

No documentation changes or additions were necessary.

All changes are either CI configuration, test helpers, or runtime code behind #if UNIFIED_NETCODE,
which cannot be defined without com.unity.netcode installed. There is no public API surface change
and no behaviour change for any current NGO user.

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

Functional Testing

Manual testing :

Manual testing done

A standalone player build containing only UnifiedNetworkTransformTest runs green locally
(Windows, Mono, 6000.7.0a5). Before the three fixes above this failed with
SetUp : Failed to start instances plus a NullReferenceException immediately after
"Starting a world for Host".

Also unverified: that com.unity.netcode: 6.7.0 resolves to the builtin copy from
manifest-unified.json on a clean checkout (locally it was added by hand to manifest.json), and that
Yamato accepts the new YAML.

Automated tests:

Covered by existing automated tests

UnifiedNetworkTransformTest already existed; this PR fixes what it was failing on and strengthens its
assertions. The new coverage is at the CI level — the test now actually runs in CI, which it never did
before, since UNIFIED_NETCODE was never defined in any existing job.

Does the change require QA team to:

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

No QA involvement requested: this is CI infrastructure plus an experimental code path that is compiled
out of every shipping configuration.

Up-port

Not needed. This PR targets develop-3.x.x directly. The unified API does not exist in NGO v2.x, and
all runtime changes are behind UNIFIED_NETCODE, which cannot be defined there.

Backports

Not needed, for the same reason — this is specific to the NGOv3.X unified work.

Adding an additional unified hybrid spawning test pass.

Fixed: - Issue with NetworkObject throwing an exception during hybrid integration test. - Issue with setting the active world prior to spawning a hybrid prefab during integration tests. - Issue with UnifiedNetcodeUpdateSystem not overriding the OnCreate method along with checks for a valid NetworkManager and/or transport within OnUpdate. - Issues (minor) with the original UnifiedNetworkTransformTest. - Adjustments to the NetcodeIntegrationTest that assures the correct active world is assigned when spawning.

Adding the modified NetcodeConfig specific to NGO. Adding the unified manifest file for running unified tests.

Unified specific settings

This "pins" the NGO version of the unified NetcodeConfig to avoid having one auto-created.

Formatting of the unified test filter.

One more time... just running the UnifiedNetworkTransformTest for this phase. A more integrated one will be coming in the next PR for this effort.

Codecov Report

All modified and coverable lines are covered by tests ✅

@@              Coverage Diff               @@
##           develop-3.x.x    #4122   +/-   ##
==============================================
  Coverage          78.01%   78.02%           
==============================================
  Files                153      153           
  Lines              26254    26260    +6     
==============================================
+ Hits               20483    20489    +6     
  Misses              5771     5771

Flag	Coverage Δ
NGOv2_project_testproject_ubuntu_pinnedTrunk	77.67% <ø> (?)

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

Files with missing lines	Coverage Δ
....netcode.gameobjects/Runtime/Core/NetworkObject.cs	75.29% <ø> (ø)

Components	Coverage Δ
com.unity.netcode.gameobjects	78.02% <100.00%> (+<0.01%)	⬆️

ℹ️ Need help interpreting these results?

Co-authored-by: Emma <emma.mcmillan@unity3d.com>

I'm not sure if we really need this part if those tests will run on each PR anyway (part of pr_code_changes_checks)

The triggers are removed in #4137

So those tests are not included by default in any other job? Like usual package tests?

This is a larger set of changes broken into a 3 part PR.
This PR runs on other PRs and on the nightly.
#4122 is the initial framework to show it working.
#4137 makes the unified integration tests work almost exactly like the CMB service test. (this still runs on PRs and Nightly)

Why we can't update our current manifest already? Would it require to move completely to testing on 6000.7?

Yeah... it would require us to remove all of the earlier editor versions.
We can do that, but we might wait until after all of the initial unified testing is done before switching over completely.

But based on yesterday's conversation we might hold off in case we decide to support 6000.3.

I'm fine with merging this as it's now and then we will adjust once we switch compleately to 6000.7

I think we already got rid of all package-lock.json so maybe we can remove this line?

Good catch.

Do we only support PlayMode tests? (otherwise it would be --suite=editor --suite=playmode)

Co-authored-by: Emma <emma.mcmillan@unity3d.com>

Removed command to delete packages-lock.json from unified tests.

From my side it's ok to merge this in the current state and then I will do a follow-up later to clean it up (once we switch to 6000.7 only testing)

…environment-var develop-3.x.x now carries PR #4122 as a single squashed commit, so the four unified files this branch also changes conflicted against their own squashed history rather than against new work. Conflict resolutions: - .yamato/project.metafile: kept this branch's deletion of unified_test_filter. The squash reintroduced it; selection is the UNIFIED_TESTS environment variable plus NetcodeIntegrationTest.UseUnifiedTests now. - .yamato/unified-tests.yml: kept this branch's version, which is the same job with the UNIFIED_TESTS variable and no --testfilter. - .yamato/_triggers.yml: took develop-3.x.x (drops a stray blank line). - NetcodeIntegrationTest.cs: took develop-3.x.x's TODO-UNIFIED tag rename. Incoming: the SceneManagementSynchronizationTests rework that distinguishes scene events from connection events and adjusts the m_UseCmbService path, which is the CMB service test fix, plus the U-PR bot config.

NoelStephensUnity added 4 commits August 16, 2026 15:02

update - ci

6a07afb

Adding an additional unified hybrid spawning test pass.

update

7547269

Adding the modified NetcodeConfig specific to NGO. Adding the unified manifest file for running unified tests.

update

319d7eb

Unified specific settings

NoelStephensUnity requested review from a team and EmandM as code owners August 16, 2026 20:26

NoelStephensUnity commented Aug 16, 2026

View reviewed changes

NoelStephensUnity added 2 commits August 16, 2026 15:51

fix

9e8b6bb

Formatting of the unified test filter.

update

93a7d71

One more time... just running the UnifiedNetworkTransformTest for this phase. A more integrated one will be coming in the next PR for this effort.

EmandM reviewed Aug 17, 2026

View reviewed changes

Apply suggestions from code review

aec318d

Co-authored-by: Emma <emma.mcmillan@unity3d.com>

NoelStephensUnity requested a review from EmandM August 19, 2026 19:25

NoelStephensUnity mentioned this pull request Aug 20, 2026

chore: unified hybrid tests enabled environment var #4137

Open

5 tasks

michalChrobot reviewed Aug 20, 2026

View reviewed changes

NoelStephensUnity and others added 3 commits August 20, 2026 08:43

Update .yamato/_triggers.yml

5afa312

Co-authored-by: Emma <emma.mcmillan@unity3d.com>

Update TODO comment for Netcode singleton issue

f848329

Update unified-tests.yml to simplify commands

3f0a970

Removed command to delete packages-lock.json from unified tests.

michalChrobot approved these changes Aug 21, 2026

View reviewed changes

Merge branch 'develop-3.x.x' into chore/enable-unified-hybrid-tests

4b32172

EmandM approved these changes Aug 21, 2026

View reviewed changes

NoelStephensUnity enabled auto-merge (squash) August 21, 2026 14:17

Merge develop-3.x.x into chore/enable-unified-hybrid-tests

ddd520a

NoelStephensUnity merged commit 82bbda0 into develop-3.x.x Aug 21, 2026
28 checks passed

NoelStephensUnity deleted the chore/enable-unified-hybrid-tests branch August 21, 2026 21:01

+              # Unified (NGO + N4E) validation, on demand.
+              # This job allows the Unified tests to be kicked off by commenting "/ci unified".
+              # This is useful for PRs where pr_code_changes_checks doesn't trigger.
+              unified_pr_checks:

+                  # alpha (unified_editors in project.metafile) rather than a supported editor, because it needs an
+                  # editor that bundles a com.unity.netcode with the unified API. Expect it to need a pin bump
+                  # whenever N4E lands breaking changes in trunk. See .yamato/unified-tests.yml.
+                  - .yamato/_run-all.yml#run_all_unified_tests

+                commands:
+                  # Swap in the manifest that pulls in com.unity.netcode (N4E). This is what causes UNIFIED_NETCODE
+                  # to be defined and therefore what makes the unified tests compile at all.
+                  - cp {{ project.path }}/Packages/manifest-unified.json {{ project.path }}/Packages/manifest.json

+                  # to be defined and therefore what makes the unified tests compile at all.
+                  - cp {{ project.path }}/Packages/manifest-unified.json {{ project.path }}/Packages/manifest.json
+                  # The committed lock was resolved against a supported editor, not the unified alpha.
+                  - rm -f {{ project.path }}/Packages/packages-lock.json

+                  - rm -f {{ project.path }}/Packages/packages-lock.json
+                  - unity-downloader-cli --fast --wait -u {{ editor }} -c Editor # Installing basic editor for tests execution
+                  - UnifiedTestRunner --testproject={{ project.path }} --suite=playmode --artifacts-path=test-results --editor-location=.Editor --testfilter="{{ unified_test_filter }}" --rerun-strategy=Test --retry={{ num_test_retries }} --clean-library-on-rerun --timeout={{ test_timeout }}

chore: enable unified hybrid tests by NoelStephensUnity · Pull Request #4122 · Unity-Technologies/com.unity.netcode.gameobjects · GitHub

Uh oh!

chore: enable unified hybrid tests - #4122

Conversation

NoelStephensUnity commented Aug 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose of this PR

This is the first part of 3 PRs:

Additional unified netcode hybrid testing details

Jira ticket

Documentation

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

Functional Testing

Up-port

Backports

Uh oh!

Choose a reason for hiding this comment

Uh oh!

codecov-github-com Bot commented Aug 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

michalChrobot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

NoelStephensUnity commented Aug 16, 2026 •
edited
Loading

codecov-github-com Bot commented Aug 16, 2026 •
edited
Loading