From f2dd86631f0a1fee25b44a9f0e5be3468ce1b60d Mon Sep 17 00:00:00 2001 From: Dean Herbert Date: Tue, 5 Aug 2025 18:50:25 +0900 Subject: [PATCH] Fix test steps potentially failing due to not waiting for database changes --- .../TestSceneBeatmapTitleWedge.cs | 22 +++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/osu.Game.Tests/Visual/SongSelectV2/TestSceneBeatmapTitleWedge.cs b/osu.Game.Tests/Visual/SongSelectV2/TestSceneBeatmapTitleWedge.cs index 4714937c8e..cc4b38b54c 100644 --- a/osu.Game.Tests/Visual/SongSelectV2/TestSceneBeatmapTitleWedge.cs +++ b/osu.Game.Tests/Visual/SongSelectV2/TestSceneBeatmapTitleWedge.cs @@ -170,8 +170,8 @@ namespace osu.Game.Tests.Visual.SongSelectV2 currentOnlineSet = onlineSet; Beatmap.Value = working; }); - AddAssert("play count = 10000", () => this.ChildrenOfType().ElementAt(0).Text.ToString() == "10,000"); - AddAssert("favourites count = 2345", () => this.ChildrenOfType().Single().Text.ToString() == "2,345"); + AddUntilStep("play count is 10000", () => this.ChildrenOfType().ElementAt(0).Text.ToString(), () => Is.EqualTo("10,000")); + AddUntilStep("favourites count is 2345", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("2,345")); AddStep("online beatmapset with local diff", () => { var (working, onlineSet) = createTestBeatmap(); @@ -181,8 +181,8 @@ namespace osu.Game.Tests.Visual.SongSelectV2 currentOnlineSet = onlineSet; Beatmap.Value = working; }); - AddAssert("play count = -", () => this.ChildrenOfType().ElementAt(0).Text.ToString() == "-"); - AddAssert("favourites count = 2345", () => this.ChildrenOfType().Single().Text.ToString() == "2,345"); + AddUntilStep("play count is -", () => this.ChildrenOfType().ElementAt(0).Text.ToString(), () => Is.EqualTo("-")); + AddUntilStep("favourites count is 2345", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("2,345")); AddStep("local beatmapset", () => { var (working, _) = createTestBeatmap(); @@ -190,8 +190,8 @@ namespace osu.Game.Tests.Visual.SongSelectV2 currentOnlineSet = null; Beatmap.Value = working; }); - AddAssert("play count = -", () => this.ChildrenOfType().ElementAt(0).Text.ToString() == "-"); - AddAssert("favourites count = -", () => this.ChildrenOfType().Single().Text.ToString() == "-"); + AddUntilStep("play count is -", () => this.ChildrenOfType().ElementAt(0).Text.ToString(), () => Is.EqualTo("-")); + AddUntilStep("favourites count is -", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("-")); } [Test] @@ -235,17 +235,17 @@ namespace osu.Game.Tests.Visual.SongSelectV2 currentOnlineSet = onlineSet; Beatmap.Value = working; }); - AddUntilStep("play count = 10000", () => this.ChildrenOfType().ElementAt(0).Text.ToString() == "10,000"); - AddUntilStep("favourites count = 2345", () => this.ChildrenOfType().Single().Text.ToString() == "2,345"); + AddUntilStep("play count is 10000", () => this.ChildrenOfType().ElementAt(0).Text.ToString(), () => Is.EqualTo("10,000")); + AddUntilStep("favourites count is 2345", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("2,345")); AddStep("click favourite button", () => this.ChildrenOfType().Single().TriggerClick()); AddStep("allow request to complete", () => resetEvent.Set()); - AddAssert("favourites count = 2346", () => this.ChildrenOfType().Single().Text.ToString() == "2,346"); + AddUntilStep("favourites count is 2346", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("2,346")); AddStep("reset event", () => resetEvent.Reset()); AddStep("click favourite button", () => this.ChildrenOfType().Single().TriggerClick()); AddStep("allow request to complete", () => resetEvent.Set()); - AddAssert("favourites count = 2345", () => this.ChildrenOfType().Single().Text.ToString() == "2,345"); + AddUntilStep("favourites count is 2345", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("2,345")); AddStep("reset event", () => resetEvent.Reset()); AddStep("click favourite button", () => this.ChildrenOfType().Single().TriggerClick()); @@ -260,7 +260,7 @@ namespace osu.Game.Tests.Visual.SongSelectV2 Beatmap.Value = working; }); AddStep("allow request to complete", () => resetEvent.Set()); - AddAssert("favourites count = 9999", () => this.ChildrenOfType().Single().Text.ToString() == "9,999"); + AddUntilStep("favourites count is 9999", () => this.ChildrenOfType().Single().Text.ToString(), () => Is.EqualTo("9,999")); AddStep("set up request handler to fail", () => {