Commit Graph

4068 Commits

Author SHA1 Message Date
vis2k
9c7c0cdd93 Weaver: separate WeavingFailed from Logging 2021-08-19 21:49:54 +08:00
vis2k
33add82042 Weaver.InvokeRpcPrefix: const instead of static state 2021-08-19 16:00:44 +08:00
vis2k
499cb72173 WeaverLists public 2021-08-19 12:51:54 +08:00
vis2k
28274f707f Weaver: WeaverLists moved into separate file 2021-08-19 12:50:44 +08:00
vis2k
625947a4cf Weaver: remove DLog and DebugEnabled. we never use it to check log files anyway. 2021-08-18 00:58:45 +08:00
vis2k
850d2bbb2b Weaver: remove unnecessary GenerateLogErrors static 2021-08-17 20:41:12 +08:00
vis2k
3c5a9de323 syntax 2021-08-17 17:18:13 +08:00
vis2k
f64b808aed syntax 2021-08-17 17:14:27 +08:00
vis2k
9a07feda28 syntax 2021-08-17 13:24:02 +08:00
vis2k
c80fbab7fa Weaver: use string interpolation instead of string.Format 2021-08-17 13:23:17 +08:00
vis2k
6a62ab9c52 Weaver: remove xml comments 2021-08-17 13:12:49 +08:00
vis2k
d0de7ed92f syntax 2021-08-17 13:09:43 +08:00
vis2k
c63152f71e Weaver: try/catch moved into Weave() to prepare reuse from ILPP 2021-08-17 13:06:49 +08:00
vis2k
2821b34398 Weaver: remove xml comments 2021-08-17 13:04:18 +08:00
vis2k
ab462a598c syntax 2021-08-17 13:01:29 +08:00
vis2k
eb3ac79bd9 Weaver: remove XML comments 2021-08-17 13:01:17 +08:00
vis2k
d329c60899 syntax 2021-08-17 12:58:20 +08:00
vis2k
d1604b5686 syntax 2021-08-17 12:57:08 +08:00
vis2k
950b439035 Weaver: remove XML comment 2021-08-17 12:56:07 +08:00
vis2k
2acc9f77ea Weaver: remove xml comments 2021-08-17 11:53:58 +08:00
vis2k
31babe6b03 Weaver: hooks moved into EntryPoint folder 2021-08-13 18:51:17 +08:00
vis2k
5d2797847e Weaver: Enter-Play-Mode check moved into separate file in Weaver folder 2021-08-13 18:45:07 +08:00
vis2k
ff34a50fdf feat: Expose NetworkEarlyUpdate/NetworkLateUpdate (see #2826) 2021-08-13 11:58:50 +08:00
vis2k
966af93e50 feat: Prevent Nested Network Managers (see #2867) 2021-08-13 11:48:41 +08:00
vis2k
8ca5203262
breaking: NetworkIdentity.spawned split into NetworkServer/Client.Spawned (#2877)
* breaking: NetworkIdentity.spawned split into NetworkServer.Spawned / NetworkClient.spawned.
WIP: 1 failing test

* clear spawned in NetworkClient.Shutdown

* MirrorTest doesn't need to clear spawned anymore. NetworkClient/NetworkServer.Shutdown already do.

* InterestManagement: use NetworkServer.spawned

* NetworkServer: use NetworkServer.spawned

* Match/SceneInterestManagement: use NetworkServer.spawned

* InterestManagementTests: use NetworkServer.spawned

* NetworkIdentity.OnStartServer: use NetworkServer.spawned

* NetworkClient: use NetworkClient.spawned everywhere; and modify OnHostClientSpawn

* NetworkIdentity.spawned: throw if server/client not active to make it more obvious why tests fail

* NetworkBehaviour GetSyncVar functions: use NetworkClient.spawned

* adjust test

* adjust test

* adjust test

* CreateNetworkedAndSpawn functions that return client object too

* adjust test

* adjust test

* adjust test

* adjust test

* adjust tests

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* adjust test

* NetworkReader.ReadNetworkIdentity adjusted
2021-08-13 11:20:32 +08:00
vis2k
a13f474243 NetworkReader.ReadNetworkBehaviour: reuse ReadNetworkIdentity 2021-08-12 18:04:20 +08:00
vis2k
4def515c9f SyncVarTest: prepare for .spawned split. this is better. 2021-08-12 15:38:39 +08:00
vis2k
cf3e7acc39 Tests: SyncVarHOokTests improved 2021-08-12 14:24:20 +08:00
MrGadget
33e77d69cb
fix: NT Base OnDrawGizmos only when playing (#2876)
* fix: NT Base OnDrawGizmos only when playing

* Update NetworkTransformBase.cs

Co-authored-by: vis2k <info@noobtuts.com>
2021-08-12 12:25:49 +08:00
vis2k
875c78c222 improve 'No writer found' error to make it more obvious why it happens and how to solve it 2021-08-12 00:18:55 +08:00
vis2k
811f1fadfc NetworkServerTests: Command tests: use host mode to prepare for easier .spawned split 2021-08-11 19:42:59 +08:00
vis2k
a7910c0c97 SyncVarTests: use host mode for easier .spawned split 2021-08-11 19:38:09 +08:00
vis2k
e22b0c0c4e NetworkWriterTests: use host mode to prepare for easier .spawned split too 2021-08-11 19:29:46 +08:00
vis2k
b089692dab Tests: NetworkTransform2 tests use host mode for now to make .spawned split easier 2021-08-11 19:22:12 +08:00
vis2k
309761a37b Tests: fix check 2021-08-11 16:32:08 +08:00
vis2k
8902cd1b4e simplify tests 2021-08-11 15:15:21 +08:00
vis2k
23cc3b8223 Tests: reuse CreateNetworkedAndSpawn 2021-08-11 15:07:02 +08:00
vis2k
c89e119743 Tests: reuse CreateNetworkedAndSpawn 2021-08-11 15:05:39 +08:00
vis2k
d9ecd21d1e Tests: reuse CreateNetworked 2021-08-11 15:03:49 +08:00
vis2k
777fb9bfb9 syntax 2021-08-11 14:23:05 +08:00
vis2k
8b42915429 Tests simplified 2021-08-11 14:22:51 +08:00
vis2k
2328747b33 remove unnecessary line 2021-08-11 14:20:44 +08:00
vis2k
06f287176c syntax 2021-08-11 14:19:34 +08:00
vis2k
47bc75cdc8 syntax 2021-08-11 14:18:40 +08:00
vis2k
1b9aaebee1 Tests simplified 2021-08-11 14:18:21 +08:00
vis2k
2fd87a49ec comments 2021-08-11 14:15:24 +08:00
vis2k
52a560ca6d Tests simplified 2021-08-11 14:14:57 +08:00
vis2k
2fff5ef02b Tests simplified 2021-08-11 14:13:59 +08:00
vis2k
ff4bc267e0 syntax 2021-08-11 14:12:14 +08:00
vis2k
f0441738d0 Tests: reuse CreateNetworked 2021-08-11 14:11:37 +08:00
vis2k
e36a9a16fb remove unnecessary destroys. CreateNetworked does it. 2021-08-11 14:10:29 +08:00
vis2k
7b62f3c7fd NetworkServerRuntimeTest: reuse CreateNetworked 2021-08-11 14:09:30 +08:00
vis2k
7c5d453a3b Tests: remove unnecessary DestroyImmediate now that CreateNetworked tracks it anyway 2021-08-11 14:04:31 +08:00
vis2k
b949034abb Tests: simplify 2021-08-11 14:03:48 +08:00
vis2k
2c1736f29e Tests: reuse CreateNetworked 2021-08-11 14:02:43 +08:00
vis2k
4f15ba9a2e syntax 2021-08-11 14:01:02 +08:00
vis2k
6c5bee8106 syntax 2021-08-11 13:59:52 +08:00
vis2k
cf29a06bca cleanup 2021-08-11 13:59:31 +08:00
vis2k
f61dd290e3 Tests: use CreateNetworked<T> 2021-08-11 13:55:27 +08:00
JesusLuvsYooh
5c45c5cd75
Defines for OnGUI (#2875)
* Defines for OnGUI KcpTransport.cs

Related to: https://github.com/vis2k/Mirror/pull/2874
"OnGUI allocates even if it does nothing. avoid in release."

* Defines for OnGUI SpatialHashing IM.cs

Related to: https://github.com/vis2k/Mirror/pull/2874
"OnGUI allocates even if it does nothing. avoid in release."

* Update SpatialHashingInterestManagement.cs

* Update KcpTransport.cs

Co-authored-by: vis2k <info@noobtuts.com>
2021-08-11 12:43:31 +08:00
vis2k
9cb4c55276 simplify tests 2021-08-11 00:34:55 +08:00
vis2k
bd6f2fd31a Runtime tests: remove old transport enabled tests. transport enabled isn't needed anymore since Unbatcher GetNextMessage 2021-08-10 19:50:02 +08:00
vis2k
e342d58e5c syntax 2021-08-10 19:49:00 +08:00
vis2k
c6bdbecfda ClientSceneTests_LocalPlayer: huge cleanup 2021-08-10 19:48:01 +08:00
vis2k
96c526a7c5 remove unnecessary TearDown 2021-08-10 19:47:56 +08:00
vis2k
38d2a7e315 syntax 2021-08-10 19:38:30 +08:00
vis2k
494484ef78 ClientSceneTest_LocalPlayer_AsHost: huge cleanup 2021-08-10 19:36:58 +08:00
vis2k
4925f1397a Tests: merge HostSetup into the only class that inherits from it 2021-08-10 19:19:14 +08:00
vis2k
87478178e9 MirrorTests: CreateNetworkedAndSpawnPlayer without <T> helper function 2021-08-10 19:11:05 +08:00
vis2k
881d90f539 Tests: remove unused FakeSpawnServerClientIdentity function 2021-08-10 19:07:06 +08:00
vis2k
080cd184c5 remove unused import 2021-08-10 19:02:41 +08:00
vis2k
47a948b87f NetworkIdentityTests: remove old HandleRpc test 2021-08-10 19:02:27 +08:00
vis2k
d094c70588 NetworkWriterTests: use CreateNetworkedAndSpawn everywhere instead of manually adding to spawned 2021-08-10 18:58:53 +08:00
vis2k
281b200a69 SyncVarHookTests: use CreateNetworkedAndSpawn everywhere instead of manually adding to spawned 2021-08-10 18:55:49 +08:00
vis2k
f146ca8b87 SyncVarTests: use CreateNetworkedAndSpawn everywhere instead of manually adding to spawned 2021-08-10 18:52:28 +08:00
vis2k
3d350f3320 GetSyncVarNetworkIdentityOnClient Test: use CreateNetworkedAndSpawn instead of manually adding to spawned 2021-08-10 18:45:07 +08:00
vis2k
4591bde5e3 GetSyncVarGameObjectOnClient Test: use CreateNetworkedAndSpawn instead of manually adding to spawned 2021-08-10 18:43:48 +08:00
JesusLuvsYooh
bafe8832f6
NTB.cs defines for OnGUI (#2874)
* NTB.cs defines for OnGUI

#if UNITY_EDITOR || DEVELOPMENT_BUILD for OnGUI() added.
Related to this post, and discord discussion: https://github.com/vis2k/Mirror/issues/2871

"OnGUI calls allocate gc even when not doing anything"
This should give developers a mini optimisation, and means no manual editing needed of NTB.cs for their final releases.

* NTB.cs defines for OnGUI + Gizmos

#if UNITY_EDITOR || DEVELOPMENT_BUILD for OnGUI() added.
Related to this post, and discord discussion: #2871

"OnGUI calls allocate gc even when not doing anything"
This should give developers a mini optimisation, and means no manual editing needed of NTB.cs for their final releases.

Will apply to the other components, if accepted/good idea.

+ Added Gizmos section by request.

* Update NetworkTransformBase.cs

Co-authored-by: vis2k <info@noobtuts.com>
2021-08-10 14:46:11 +08:00
ninjakickja
ae622bfa57
Changes to validation of catchupThreshold - NetworkTransform2k - NetworkTransformBase (#2853)
* Changes to validation of catchupThreshold.

catchupThreshold needs to be at least 4, or it will interfere/conflict with checking if we have at least 3 old enough buffers. Catchup will decrease buffer while check needs at least 3 older.
catchupThreshold should also at least be bufferTimeMultiplier + 2, to take into account of the first 2 snapshots which is used for interpolation, before trying to implement catch up.

* Changes to validation of catchupThreshold - NetworkTransform2k - NetworkTransformBase

catchupThreshold needs to be at least 4, or it will interfere/conflict with checking if we have at least 3 old enough buffers. Catchup will decrease buffer while check needs at least 3 older.
catchupThreshold should also at least be bufferTimeMultiplier + 2, to take into account of the first 2 snapshots which is used for interpolation, before trying to implement catch up.

* Changes to validation of catchupThreshold - NetworkTransform2k - NetworkTransformBase

catchUpThreshold should be a minimum of bufferTimeMultiplier + 3, to prevent clashes with SnapshotInterpolation looking for at least 3 old enough buffers, else catch up will be implemented while there is not enough old buffers, and will result in jitter.

* Update NetworkTransformBase.cs

Co-authored-by: vis2k <info@noobtuts.com>
2021-08-10 14:25:37 +08:00
vis2k
79c9d2cc6a Tests simplified 2021-08-09 16:56:43 +08:00
vis2k
6031f8936c Tests simplified 2021-08-09 16:47:53 +08:00
vis2k
d02262e4a7 Test simplified 2021-08-09 16:46:39 +08:00
vis2k
c77406730c NetworkServerTests: SendCommand test simplified 2021-08-09 16:45:42 +08:00
vis2k
0340dd2d38 NetworkServerTests: SendCommand test simplified 2021-08-09 16:44:44 +08:00
vis2k
b99b6d314d NetworkServerTests: SendCommand test simplified 2021-08-09 16:43:42 +08:00
vis2k
b340397ad0 NetworkServerTests: SendCommand test simplified 2021-08-09 16:42:48 +08:00
vis2k
4f72d5f8f6 NetworkServer.CleanupNetworkIdentities renamed to CleanupSpawned because that's what it does. 2021-08-09 16:29:08 +08:00
MrGadget1024
d4f7a26278 Removed Network Observer Script Template 2021-08-09 00:32:26 -04:00
MrGadget1024
279db96a16 Consistant naming of NetworkManager object in example scenes 2021-08-09 00:06:43 -04:00
MrGadget1024
ed291183f6 fix: Multiple Matches example transport to KCP 2021-08-09 00:01:53 -04:00
MrGadget1024
9aced8f106 fix: Updated Multiple Matches example to use Match Int. Mgmt. 2021-08-08 23:53:08 -04:00
MrGadget1024
61f2ab809d Suppress noisy Debug 2021-08-08 12:56:57 -04:00
MrGadget1024
43a4fdc4be Added HelpUrl 2021-08-08 11:47:11 -04:00
vis2k
6d59807745 add logs 2021-08-08 18:46:22 +08:00
vis2k
f512928a3d comment 2021-08-08 18:36:20 +08:00
vis2k
8f8a8ce690 Test simplified 2021-08-08 18:34:57 +08:00
vis2k
f88c3af277 syntax 2021-08-08 18:34:09 +08:00
vis2k
046e6d45f7 NetworkIdentitySerializationTests: simplify 2021-08-08 18:33:02 +08:00
vis2k
d7ee83d696 formatting 2021-08-08 18:31:24 +08:00
vis2k
b3bbec793f NetworkIdentitySerializationTests: SerializationException 2021-08-08 18:29:21 +08:00
vis2k
d0ea26b42f shorter name 2021-08-08 18:23:19 +08:00
vis2k
de4d2dc92f Tests: NetworkIdentity serialization tests moved to separate file 2021-08-08 18:22:32 +08:00
vis2k
6ed56f08ea Tests: CreateNetworkedAndSpawn<T,U,V> helper 2021-08-08 18:15:29 +08:00
vis2k
baad907fe8 Tests: CreateNetworkedAndSpawn<T,U> helper 2021-08-08 18:15:26 +08:00
vis2k
26fd44c0bd MIRROR_44_0_OR_NEWER 2021-08-08 17:40:26 +08:00
vis2k
e28d7e903a NetworkIdentity.OnSerializeSafely: remove unnecessary ownerWritten, observersWritten parameters 2021-08-08 13:59:09 +08:00
vis2k
6ce8dfcc23 syntax 2021-08-08 13:41:54 +08:00
vis2k
1e89a3a65d NetworkServer.CreateSpawnMessagePayload: don't need OnSerializeAllSafely 'ownerWritten' and 'observersWritten' values. always convert to ArraySegment - and if nothing was written, they are empty. 2021-08-08 13:34:00 +08:00
vis2k
9dd1420edf NetworkWriterTest: empty arraysegment 2021-08-08 13:29:13 +08:00
vis2k
a76fa59fee NetworkIdentity.OnSerializeSafely simplified: owner/observers 'written' changed from int to bool 2021-08-08 13:23:39 +08:00
vis2k
adc7687807 stepsize 10 is enough 2021-08-07 22:09:34 +08:00
vis2k
a2f5659723 comment 2021-08-07 22:01:35 +08:00
vis2k
93246ffda5 better logging 2021-08-07 21:51:26 +08:00
vis2k
59b7d79db6 rename to make it more obvious 2021-08-07 21:47:24 +08:00
vis2k
6e87337792 Quaternion Compression Tests: [0..360] in 5 degree steps for all x, y, z components 2021-08-07 21:38:24 +08:00
vis2k
1666bf5309 syntax 2021-08-07 21:31:10 +08:00
vis2k
572ce6c751 LargestAbsoluteComponentIndex: rename largest to largestAbs to be more obvious 2021-08-07 21:27:38 +08:00
vis2k
2aa791a726 fix: #2674 - Quaternion Compression LargestAbsoluteComponentIndex largest absolute was accidentally initialized with largest, instead of largest absolute 2021-08-07 21:27:18 +08:00
vis2k
c5080d1ab3 Tests: LargestAbsoluteComponentIndex test to reproduce #2674 2021-08-07 21:24:10 +08:00
vis2k
3d67c72336 enable test 2021-08-07 21:17:10 +08:00
vis2k
13c2a42a77 remove unused runtime SceneObjectSpawningTests 2021-08-07 19:31:29 +08:00
vis2k
99109076e4 syntax 2021-08-07 19:27:23 +08:00
vis2k
300e236715 syntax 2021-08-07 19:26:38 +08:00
vis2k
1fc008f1ef syntax 2021-08-07 19:24:43 +08:00
vis2k
02cc3de7b8 SnapshotInterpolationTests: casting not necessary anymore 2021-08-07 19:23:07 +08:00
vis2k
664f21ba45 RemoteTestBase: reuse ConnectHostClientBlockingAuthenticatedAndReady 2021-08-07 19:21:36 +08:00
vis2k
964188868d syntax 2021-08-07 19:20:31 +08:00
vis2k
8673471cf1 comment 2021-08-07 19:20:11 +08:00
vis2k
0fdb39fa1c CreateNetworkedAndSpawn without component helper function 2021-08-07 19:15:01 +08:00
vis2k
aff28a0c07 syntax 2021-08-07 19:12:09 +08:00
vis2k
e3f7b8ce74 Tests: rewrite #2629 fix test 2021-08-07 18:56:42 +08:00
vis2k
f12082d0a3 Tests: CreateNetworkedAndSpawn remove unnecesary .connetionToServer setup. it's not necessary anymore. 2021-08-07 18:50:31 +08:00
vis2k
b29a76d242 add assert just to be sure 2021-08-07 18:45:28 +08:00
vis2k
e52b9b24de fix comment 2021-08-07 18:44:52 +08:00
vis2k
4436fb85ad syntax 2021-08-07 15:18:04 +08:00
vis2k
e2a26cd438 syntax 2021-08-07 15:16:46 +08:00
vis2k
ed505b7b65 syntax 2021-08-07 15:16:11 +08:00
vis2k
95a7899b76 TODO 2021-08-07 15:13:22 +08:00
vis2k
90674529e0 syntax 2021-08-07 15:12:23 +08:00
vis2k
fd8023dafe remove redundant TargetRpc (manual) test 2021-08-07 15:11:24 +08:00
vis2k
39ebfc6126 remove redundant RPC (manual) test 2021-08-07 14:04:21 +08:00
vis2k
b71e2f373a syntax 2021-08-07 13:59:35 +08:00
vis2k
98eba5c9c1 syntax 2021-08-07 13:54:18 +08:00
vis2k
f74fb388b4 syntax 2021-08-07 13:45:42 +08:00
vis2k
3350760edb Multiplex Test: reuse CreateGameObject with component 2021-08-07 13:44:52 +08:00
vis2k
70f117a950 AOI Test: reuse CreateGameObject with component 2021-08-07 13:40:46 +08:00
vis2k
dccf1a95f1 AOI Test: reuse CreateGameObject with component 2021-08-07 13:39:43 +08:00
vis2k
5dad5db819 Tests: CreateGameObject with Component helper 2021-08-07 13:39:32 +08:00
vis2k
0888776f06 AOI Test: use CreateGameObject for automated tracking & destroying 2021-08-07 13:37:47 +08:00
vis2k
5c8bd25b7e namespaces 2021-08-07 13:30:11 +08:00