fix: ChangeOwnership sending own clientId and added NetworkSceneHandle struct

[EmandM]

Purpose of this PR

• Update the ownership permissions tests to work with the CMB service.
• Fix bug where in DA mode when changing ownership the client would send themselves in the ClientIds list
• Added the NetworkSceneHandle struct to wraps the SceneManagement.Scene.handle representation.
  • The UnityEngine are changing the handle from an int to a SceneHandle struct.
  • They are also deprecating automatic conversion between the int and the SceneHandle
  • The NetworkSceneHandle struct allows us to hide this change from the rest of the codebase. It may also enable us to refactor our scene management as it gives us a more powerful representation of a scene.

Jira ticket

MTTB-1570
MTTB-1646

Changelog

• Changed: Fixed the OwnershipPermissionsTests to work with the new server-independent test framework
• Added: A NetworkSceneHandle struct that wraps the SceneManagement.Scene.handle representation. This allows us to work smoothly with the latest functionality of scene management in the UnityEngine

Documentation

• No documentation changes or additions were necessary.

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

We need to playtest with the SceneLoading scene to ensure that's still all working as expected.

Functional Testing

Manual testing :

• Manual testing done

Automated tests:

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

Does the change require QA team to:

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

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

This test is specific to 2.x so no backport required.

[michalChrobot]

Out of curiosity, why 6000.4? Similar question to define above, why 6000.3

[EmandM]

Entities broke us 🥲 in 6.3 they introduced a struct instead of an int for the scene handle. and then in 6.4 they removed the implicit conversions between the struct and the int.

[NoelStephensUnity]

Great job narrowing down the core issue with the more recent SceneHandle updates in v6.3 with a nice wrapper class, NetworkSceneHandle, to provide a smooth experience for users!
🥇

In the name of "getting the PR queue uncongested", the changelog entry for this can be found in #3707 to avoid running through CI just for the entry update.