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
vis2k
4bc1c3eb20
NetworkIdentityTests HandleEvent tests invalid component index / function hash too
2020-03-03 13:19:09 +01:00
vis2k
bbcc3599ef
NetworkIdentityTests HandleRpc tests invalid component index / function hash too
2020-03-03 13:19:03 +01:00
vis2k
b9dab2ca7a
NetworkIdentityTests HandleCommand tests invalid component index / function hash too
2020-03-03 13:18:55 +01:00
vis2k
ea54890dca
NetworkIdentityTests: HandleCommand/Rpc/SyncEvent tests improved
2020-03-03 13:15:54 +01:00
vis2k
6cdb9a3d8b
NetworkIdentityTests: HandleSyncEvent
2020-03-03 13:08:58 +01:00
vis2k
0f362ec1cf
NetworkIdentityTests: HandleRPC
2020-03-03 13:06:20 +01:00
vis2k
598766764c
typo
2020-03-03 13:03:25 +01:00
vis2k
0b257fbd3a
NetworkIdentityTests: HandleCommand
2020-03-03 13:02:44 +01:00
vis2k
bd9747e735
NetworkReader/WriterTest: benchmark doesn't need to run each time, takes at least 6s together.
2020-03-03 12:15:07 +01:00
uwee
67462553fb
test add NetworkManagerTests ( #1537 )
...
* test add NetworkManagerTests
not complete and 1 does not reliably work
* fix broken client test
* remove incomplete test for NG
* added more state checking
* shutdown state test
* add startposition tests
2020-03-03 09:28:28 +01:00
Lymdun
ca166bd95f
Improve LocalConnectionTest ( #1538 )
2020-03-02 22:41:27 +01:00
