vis2k
342f5b8b98
NetworkBehaviourTests: SendCommandInternal
2020-03-05 21:43:31 +01:00
vis2k
85dab627ef
NetworkIdentityTests: clear up delegates
2020-03-05 21:43:14 +01:00
vis2k
b91ed1aead
NetworkServerTests: clear up delegates
2020-03-05 21:43:10 +01:00
vis2k
1bdd6a80b2
NetworkBehaviourTests: SendCommandInternal improved
2020-03-05 16:01:42 +01:00
vis2k
faa78493b9
NetworkBehaviourTests: SendCommandInternal
2020-03-05 15:47:26 +01:00
vis2k
b330a9c6b3
accidentally merged this one
2020-03-05 14:31:12 +01:00
vis2k
e37e2ff2fc
NetworkBehaviourTests: InitSyncObject
2020-03-05 13:42:45 +01:00
vis2k
cdea476212
NetworkBehaviourTests: HookGuard
2020-03-05 13:40:26 +01:00
vis2k
c4060df00b
NetworkBehaviourTests: OnCheckObserverTrueByDefault
2020-03-05 13:26:24 +01:00
vis2k
43909bb40e
NetworkBehaviourTests: ComponentIndex
2020-03-05 13:20:31 +01:00
vis2k
eb977896f8
NetworkIdentityTests: HasIdentitysConnectionToClient
2020-03-05 12:59:04 +01:00
vis2k
892215cfe3
NetworkIdentityTests: HasIdentitysConnectionToServer
2020-03-05 12:58:55 +01:00
vis2k
d396857148
NetworkBehaviourTests: HasIdentitysNetId
2020-03-05 12:42:03 +01:00
vis2k
6fffb051d1
NetworkBehaviourTests: HasNoAuthorityByDefault
2020-03-05 12:24:39 +01:00
vis2k
f326520d31
NetworkBehvaviourTests: simplify
2020-03-05 12:21:29 +01:00
vis2k
be6696032f
NetworkIdentityTests: IsClientOnly
2020-03-05 12:19:48 +01:00
vis2k
8f9064a923
NetworkBehaviourTests: IsServerOnly
2020-03-05 10:09:08 +01:00
vis2k
15915f573e
NetworkBehaviourTests preparations
2020-03-04 23:54:22 +01:00
vis2k
1d120a9d80
NetworkIdentity.RebuildObserver: move observers code above host visibility code
2020-03-04 18:38:17 +01:00
vis2k
d9ea5fdcf2
NetworkIdentityTests: RebuildObserversReturnsIfNull
2020-03-04 17:21:56 +01:00
vis2k
37e783f625
NetworkIdentityTests: RebuildObserversAddRemoveAndVisListTest that tests a full add/remove observers workflow
2020-03-04 17:17:21 +01:00
vis2k
0093a2cfe1
NetworkConnection.visList made internal so we can test RebuildObservers fully. It's a complex function and we need to be 100% sure that visList behaviour is correct.
2020-03-04 17:16:59 +01:00
vis2k
290561167d
NetworkIdentityTests: RebuildObserversAddsReadyComponentConnectionsIfImplemented
2020-03-04 14:55:34 +01:00
vis2k
a5f495a774
fix: NetworkIdentity.RebuildObservers: added missing null check for observers coming from components that implement OnRebuildObservers. Previously this caused a NullReferenceException.
2020-03-04 14:55:27 +01:00
vis2k
26c0c03fd8
NetworkIdentityTests: RebuildObserversSetsHostVisibility
2020-03-04 14:34:07 +01:00
vis2k
1ddb9ae075
add comment
2020-03-04 13:21:18 +01:00
vis2k
0f110eb4b7
NetworkIdentity.RebuildObservers: put conn null/ready checks into one line
2020-03-04 13:20:39 +01:00
vis2k
4ba456439a
NetworkIdentity.RebuildObservers: improve comments to explain what happens
2020-03-04 13:15:56 +01:00
vis2k
dbfd1830d3
NetworkIdentity.RebuildObservers: syntax improved
2020-03-04 13:14:26 +01:00
vis2k
1133fe4e25
forgot to remove comment
2020-03-04 09:23:23 +01:00
vis2k
8e6f5ba89c
NetworkIdentityTests: RebuildObserversDoesNotAddServerConnectionsIfImplemented
2020-03-04 09:22:04 +01:00
vis2k
9c09c26a5c
fix: SendToObservers missing result variable
2020-03-04 09:20:28 +01:00
vis2k
66d858e631
NetworkIdentityTests: RebuildObserversAddsReadyServerConnectionsIfNotImplemented
2020-03-04 09:17:47 +01:00
Paul Pacheco
9c81c9b2eb
refactor: SendToObserver returns void ( #1539 )
...
We calculate a bool out of SendToObserver, but it is not clear what it is
and we don't use the result at all. So just simplify things.
2020-03-04 08:52:17 +01:00
vis2k
549c729a07
NetworkIdentityTests: AddAllReadyServerConnectionsToObservers
2020-03-03 23:43:20 +01:00
vis2k
a5602011dd
rename to be 100% obvious
2020-03-03 23:31:44 +01:00
vis2k
66d3e8ddf7
NetworkIdentity.RebuildObservers code split into AddAllServerConnectionsToObservers helper function
2020-03-03 23:29:47 +01:00
vis2k
d6a93275f9
NetworkIdentity.RebuildObservers: result renamed to rebuildOverwritten because that's what it's for
2020-03-03 23:22:49 +01:00
vis2k
6eedd2405c
NetworkIdentity.GetNewObservers: result renamed to rebuildOverwritten because that's what it's for
2020-03-03 23:22:37 +01:00
vis2k
15a47ca19c
NetworkIdentityTests: RebuildObserversOnlyAddsOwnPlayerIfReady
2020-03-03 23:05:41 +01:00
vis2k
3e8c6089b0
NetworkIdentityTests: RebuildObserversAddsOwnReadyPlayer
2020-03-03 23:05:41 +01:00
Paul Pacheco
60226797ac
tests: Fix tests
...
CI/CD pipeline runs in headless mode, so this check is invalid
2020-03-03 15:43:25 -06:00
Paul Pacheco
108c83a43a
refactor: use NUnit idiomatic constraints
...
Why?
Suppose you have this:
```cs
Assert.That(manager.networkAddress == "localhost");
```
and suppose it fails, it will give you:
"Expected True, but got False"
which does not give you anything useful.
Suppose you have this instead:
```cs
Assert.That(manager.networkAddress, Is.EqualTo("localhost"));
```
Suppose it fails, now it will give you a much more useful error message:
"Expected 'localhost' but got 'xxxxx'"
It makes debugging tests a lot easier.
2020-03-03 15:39:25 -06:00
vis2k
fb9c55bf6e
NetworkIdentityTests cleaned up
2020-03-03 22:37:13 +01:00
vis2k
b2a094930f
syntax
2020-03-03 22:27:27 +01:00
vis2k
a302306dd1
NetworkIdentityTests: GetNewObserversFalseIfNoComponents test
2020-03-03 22:24:13 +01:00
vis2k
102484b811
NetworkIdentityTests: GetNewObserversClearsHashSet test
2020-03-03 22:24:13 +01:00
vis2k
260ca9a491
NetworkIdentityTests: GetNewObservers test
2020-03-03 22:24:13 +01:00
vis2k
01f5fb4a94
NetworkIdentity.RebuildObservers: GetNewObservers code split into separate function
2020-03-03 22:24:13 +01:00
Paul Pacheco
e00e8195bb
Simplify
2020-03-03 10:40:57 -06:00
