mirror of https://github.com/ppy/osu
Fix chat overlay tests
Not entirely sure why they were failing previously, but the most likely explanation is that by freak accident some mock requests would previously execute immediately rather than be scheduled on the API thread, which would change execution ordering and ensure that `ChannelManager.CurrentChannel` would become the joined channel, rather than remaining at the channel listing.
This commit is contained in:
parent
8ffd4aa82c
commit
8b04455c29
|
@ -446,7 +446,7 @@ public void TestKeyboardCloseAndRestoreChannel()
|
|||
{
|
||||
AddStep("Show overlay with channel 1", () =>
|
||||
{
|
||||
channelManager.JoinChannel(testChannel1);
|
||||
channelManager.CurrentChannel.Value = channelManager.JoinChannel(testChannel1);
|
||||
chatOverlay.Show();
|
||||
});
|
||||
waitForChannel1Visible();
|
||||
|
@ -462,7 +462,7 @@ public void TestKeyboardNewChannel()
|
|||
{
|
||||
AddStep("Show overlay with channel 1", () =>
|
||||
{
|
||||
channelManager.JoinChannel(testChannel1);
|
||||
channelManager.CurrentChannel.Value = channelManager.JoinChannel(testChannel1);
|
||||
chatOverlay.Show();
|
||||
});
|
||||
waitForChannel1Visible();
|
||||
|
|
Loading…
Reference in New Issue