Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: update universal rpc tests revert spawnmany timeout #3168

Conversation

NoelStephensUnity
Copy link
Collaborator

Reverting the change in timeout for WhenManyObjectsAreSpawnedAtOnce_AllAreReceived.
Using NGO v2 version of:

  • UniversalRpcTestSendingWithGroupOverride
  • UniversalRpcTestSendingWithGroupNotOverride

And bumping the timeout to 20 minutes.

Changelog

NA

Testing and Documentation

  • No tests have been added.
  • Includes unit tests.
  • Includes integration tests.
  • No documentation changes or additions were necessary.
  • Includes documentation for previously-undocumented public API entry points.
  • Includes edits to existing public API documentation.

reverting the removal of the timeout.
making it actually 6 minutes.
Using the NGO v2. version of the UniversalRpcTestSendingWithGroupNotOverride and TestSendingWithGroupOverride tests.
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner December 12, 2024 06:16
@NoelStephensUnity NoelStephensUnity merged commit 5425071 into release/1.12.0 Dec 12, 2024
5 of 23 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/update-universal-rpc-tests-revert-spawnmany-timeout branch December 12, 2024 06:19
michalChrobot pushed a commit that referenced this pull request Dec 12, 2024
* update

reverting the removal of the timeout.
making it actually 6 minutes.

* fix

Using the NGO v2. version of the UniversalRpcTestSendingWithGroupNotOverride and TestSendingWithGroupOverride tests.
michalChrobot added a commit that referenced this pull request Dec 15, 2024
)

* Added setup necessary for release automation of NGO (#3141)

* Corrected license file

* Removed old files not conforming to pvp rules

* Corrected typo in changelog

* Added samples description to package.json while removing deprecated .samples.json

* Inclusion of CI generation files

* Changed Samples folder name to Samples~

* Added path field in package.json for samples

* fix: removing timeouts from integration tests (#3167)

Removing timesouts from our integration tests:
- TestSendingWithGroupOverride
- TestSendingWithGroupNotOverride
- WhenManyObjectsAreSpawnedAtOnce_AllAreReceived(

* fix: update universal rpc tests revert spawnmany timeout (#3168)

* fix: adding extended timeout to all universal rpc tests (#3170)

Adding extended timeout to all universal rpc tests

* Extended timeout for validation jobs on editors lower than 6000 to 5h (#3173)

---------

Co-authored-by: Noel Stephens <[email protected]>
NoelStephensUnity added a commit that referenced this pull request Jan 10, 2025
* update

incrementing the version number and updating changelog to match version number.

* Added setup necessary for release automation of NGO (#3141)

generated files for release automation within Unity org

* fix: networklist editor memory leak [backport] (#3148)

* fix

This fixes the issue with NetworkLists on in-scene placed NetworkObjects causing small memory leaks when entering and exiting playmode.

* update

adding changelog entry

* update

adding PR number to log entry

* Corrected setup necessary for release automation of NGO (#3149)

* generated files for release automation within Unity org

* Corrected pvp profiles check

* Chore/phase 1 api documentation update 1 12 0 (#3158)

* Update README.md

Adding note about where to find the NGO v2 branch along with the examples.

* chore: api documentation phase 1 [backport] (#3157)

* update

updating XML API documentation for ConnectionEvent and ConnectionEventData.

* update

Updating XML API documentation for RpcTargetUse.

* Update com.unity.netcode.gameobjects/Runtime/Messaging/RpcTargets/RpcTarget.cs

Co-authored-by: Emma <[email protected]>

* Update com.unity.netcode.gameobjects/Runtime/Messaging/RpcTargets/RpcTarget.cs

Co-authored-by: Emma <[email protected]>

---------

Co-authored-by: Emma <[email protected]>

---------

Co-authored-by: Emma <[email protected]>

* PVP and Wrench fixes

* Updated wrench version to 0.10.30

* Corrected license file

* Removed old files not conforming to pvp rules

* Corrected typo in changelog

* Added samples description to package.json while removing deprecated .samples.json

* Inclusion of CI generation files

* Pvp fixes for automated release

* Changed Samples folder name to Samples~

* Added path field in package.json for samples

* Increased wrench timeout for editors below 6000

* Increasing wrench timeout to 2h for wrench validation jobs

* Timeout of validation jobs increased to 2h for editors below 6000

* fix: removing timeouts from integration tests (#3167)

fix

Removing timesouts from our integration tests:
- TestSendingWithGroupOverride
- TestSendingWithGroupNotOverride
- WhenManyObjectsAreSpawnedAtOnce_AllAreReceived(

* fix: update universal rpc tests revert spawnmany timeout (#3168)

* update

reverting the removal of the timeout.
making it actually 6 minutes.

* fix

Using the NGO v2. version of the UniversalRpcTestSendingWithGroupNotOverride and TestSendingWithGroupOverride tests.

* fix (#3169)

ignoring the two failing tests.

* fix: adding extended timeout to all universal rpc tests (#3170)

fix

Adding extended timeout to all universal rpc tests

* Extended timeout for validation jobs on editors lower than 6000 to 5h (#3173)

* update

Adding change log entry for this PR.

* update

removing changelog entry as that belong to a different PR.

---------

Co-authored-by: Michał Chrobot <[email protected]>
Co-authored-by: Emma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant