Fix scope being disposed prematurely

Somehow this didn't happen in .NET 7, but obviously the code is wrong here. A classic case of using and then not awaiting the task, leading to a race condition. I suppose this is a follow-up on 6cebeb1
Sebazzz · Nov 15, 2023 · 0723e45 · 0723e45
1 parent af1360e
commit 0723e45
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/tests/Return.Web.Tests.Integration/Pages/JoinRetrospectiveTests.cs b/tests/Return.Web.Tests.Integration/Pages/JoinRetrospectiveTests.cs
@@ -1,6 +1,6 @@
 // ******************************************************************************
 //  © 2019 Sebastiaan Dammann | damsteen.nl
-// 
+//
 //  File:           : JoinRetrospectiveTests.cs
 //  Project         : Return.Web.Tests.Integration
 // ******************************************************************************
@@ -181,9 +181,9 @@ public sealed class JoinRetrospectiveTests : PageFixture<JoinRetrospectivePage>
         Assert.That(() => secondInstance.OnlineList.OnlineListItems.AllInnerTextsAsync().GetAwaiter().GetResult(), Has.One.Contains(myName));
     }
 
-    private Task SetRetrospective(string retroId, Action<Retrospective> action) {
+    private async Task SetRetrospective(string retroId, Action<Retrospective> action) {
         using IServiceScope scope = this.App.CreateTestServiceScope();
-        return scope.SetRetrospective(retroId, action);
+        await scope.SetRetrospective(retroId, action);
     }
 
     private async Task<string> CreateRetrospective(string facilitatorPassword, string password) {

diff --git a/tests/Return.Web.Tests.Integration/Pages/RetrospectiveLobbyTestsBase.cs b/tests/Return.Web.Tests.Integration/Pages/RetrospectiveLobbyTestsBase.cs
@@ -1,6 +1,6 @@
 // ******************************************************************************
 //  ©  Sebastiaan Dammann | damsteen.nl
-// 
+//
 //  File:           : RetrospectiveLobbyTestsBase.cs
 //  Project         : Return.Web.Tests.Integration
 // ******************************************************************************
@@ -67,9 +67,9 @@ public class RetrospectiveLobbyTestsBase : TwoClientPageFixture<RetrospectiveLob
 
     protected Task WaitNavigatedToLobby() => Task.WhenAll(WaitNavigatedToLobby(this.Client1), WaitNavigatedToLobby(this.Client2));
 
-    protected Task SetRetrospective(Action<Retrospective> action) {
+    protected async Task SetRetrospective(Action<Retrospective> action) {
         using IServiceScope scope = this.App.CreateTestServiceScope();
-        return scope.SetRetrospective(this.RetroId, action);
+        await scope.SetRetrospective(this.RetroId, action);
     }
 
     private static async Task WaitNavigatedToLobby(RetrospectiveLobby pageObject) {