Increase default LSP notification test timeouts

YarnSpinnerTool · Sep 18, 2023 · a21622f · a21622f
1 parent 0b1b92e
commit a21622f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/YarnSpinner.LanguageServer.Tests/LanguageServerTestsBase.cs b/YarnSpinner.LanguageServer.Tests/LanguageServerTestsBase.cs
@@ -152,7 +152,7 @@ protected async Task<T> GetTaskResultOrTimeoutAsync<T>(TaskCompletionSource<T> t
                         CancellationToken
                     )
                 );
-                task.Task.Should().BeSameAs(winner, "because the result should arrive within {0} seconds", timeout);
+                winner.Should().BeSameAs(task.Task, "because the result should arrive within {0} seconds", timeout);
 
                 return await task.Task;
             }
@@ -172,7 +172,7 @@ protected async Task<T> GetTaskResultOrTimeoutAsync<T>(TaskCompletionSource<T> t
         /// <param name="timeout">The amount of time to wait for
         /// diagnostics.</param>
         /// <returns>A collection of <see cref="Diagnostic"/> objects.</returns>
-        protected async Task<PublishDiagnosticsParams> GetDiagnosticsAsync(Func<PublishDiagnosticsParams, bool>? test = null, double timeout = 2f)
+        protected async Task<PublishDiagnosticsParams> GetDiagnosticsAsync(Func<PublishDiagnosticsParams, bool>? test = null, double timeout = 5f)
         {
             return await GetTaskResultOrTimeoutAsync(
                 ReceivedDiagnosticsNotifications.AddListener(test) , 
@@ -181,7 +181,7 @@ protected async Task<PublishDiagnosticsParams> GetDiagnosticsAsync(Func<PublishD
             );
         }
 
-        protected async Task<NodesChangedParams> GetNodesChangedNotificationAsync(Func<NodesChangedParams, bool>? test = null, double timeout = 2f) {
+        protected async Task<NodesChangedParams> GetNodesChangedNotificationAsync(Func<NodesChangedParams, bool>? test = null, double timeout = 5f) {
             return await GetTaskResultOrTimeoutAsync(
                 NodesChangedNotification.AddListener(test), 
                 null,