charleslemaux/Mirror
1
0
Fork 0
mirror of https://github.com/MirrorNetworking/Mirror.git synced 2024-11-18 02:50:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

NetworkIdentityTests: AssignAndRemoveClientAuthority test improved

Browse Source
This commit is contained in:
vis2k 2020-03-02 15:50:32 +01:00
parent eb958ae182
commit 4b6affb523
1 changed files with 22 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
Assets/Mirror/Tests/Editor/NetworkIdentityTests.cs
View File

@ -604,12 +604,6 @@ public void AssignAndRemoveClientAuthority()
callbackState = state;
};
// we can only handle authority on the server.
// start the server so that isServer is true.
Transport.activeTransport = Substitute.For<Transport>(); // needed in .Listen
NetworkServer.Listen(1);
Assert.That(identity.isServer, Is.True);
// create a connection
ULocalConnectionToClient owner = new ULocalConnectionToClient();
owner.isReady = true;
@ -620,8 +614,22 @@ public void AssignAndRemoveClientAuthority()
{ MessagePacker.GetId<SpawnMessage>(), (msg => ++spawnCalled) }
});
// assign authority
// assigning authority should only work on server.
// if isServer is false because server isn't running yet then it
// should fail.
LogAssert.ignoreFailingMessages = true; // error log is expected
bool result = identity.AssignClientAuthority(owner);
LogAssert.ignoreFailingMessages = false;
Assert.That(result, Is.False);
// we can only handle authority on the server.
// start the server so that isServer is true.
Transport.activeTransport = Substitute.For<Transport>(); // needed in .Listen
NetworkServer.Listen(1);
Assert.That(identity.isServer, Is.True);
// assign authority
result = identity.AssignClientAuthority(owner);
Assert.That(result, Is.True);
Assert.That(identity.connectionToClient, Is.EqualTo(owner));
Assert.That(callbackCalled, Is.EqualTo(1));
@ -643,6 +651,13 @@ public void AssignAndRemoveClientAuthority()
Assert.That(identity.connectionToClient, Is.EqualTo(owner));
Assert.That(callbackCalled, Is.EqualTo(1));
// someone might try to remove authority by assigning null.
// make sure this fails.
LogAssert.ignoreFailingMessages = true; // error log is expected
result = identity.AssignClientAuthority(null);
LogAssert.ignoreFailingMessages = false;
Assert.That(result, Is.False);
// removing authority for the main player object shouldn't work
owner.identity = identity; // set connection's player object
LogAssert.ignoreFailingMessages = true; // error log is expected