1
0
mirror of https://github.com/ppy/osu.git synced 2025-02-13 10:33:07 +08:00

Remove arbitrarily-set wait step with until step instead, keeping the comment

This commit is contained in:
Salman Ahmed 2021-08-13 15:06:11 +03:00
parent 7cf6b551d3
commit c32ba9e38f

View File

@ -95,10 +95,8 @@ namespace osu.Game.Tests.Visual.Multiplayer
AddUntilStep("wait for players to load", () => spectatorScreen.AllPlayersLoaded);
// components wrapped in skinnable target containers load asynchronously, potentially taking more than one frame to load.
// wait once to properly execute the assert.
AddWaitStep("wait for async load", 1);
AddAssert("all interactive elements removed", () => this.ChildrenOfType<Player>().All(p =>
// therefore use until step rather than direct assert to account for that.
AddUntilStep("all interactive elements removed", () => this.ChildrenOfType<Player>().All(p =>
!p.ChildrenOfType<PlayerSettingsOverlay>().Any() &&
!p.ChildrenOfType<HoldForMenuButton>().Any() &&
!p.ChildrenOfType<SongProgressBar>().Single().ShowHandle));