notifications should not cancel (#328)

david-driscoll · web-flow · commit 4fbe2b5571eb · 2020-08-23T13:25:57.000-04:00
* notifications should not cancel, instead they should complete in order as expected (as they are typically fire and forget anyway)

* some requests should not be parallel that are

* fixed failing tests
diff --git a/src/JsonRpc/InputHandler.cs b/src/JsonRpc/InputHandler.cs
@@ -562,8 +562,6 @@ private SchedulerDelegate RouteNotification(IRequestDescriptor<IHandlerDescripto
             (contentModifiedToken, scheduler) =>
                 // ITS A RACE!
                 Observable.Amb(
-                    contentModifiedToken
-                       .Do(_ => { _logger.LogTrace("Notification was abandoned due to content be modified"); }),
                     Observable.Timer(_requestTimeout, scheduler)
                               .Select(z => Unit.Default)
                               .Do(
diff --git a/src/Protocol/Document/IDocumentFormattingHandler.cs b/src/Protocol/Document/IDocumentFormattingHandler.cs
@@ -8,7 +8,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Document
 {
-    [Serial]
+    [Parallel]
     [Method(TextDocumentNames.DocumentFormatting, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Document/IDocumentOnTypeFormattingHandler.cs b/src/Protocol/Document/IDocumentOnTypeFormattingHandler.cs
@@ -8,7 +8,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Document
 {
-    [Serial]
+    [Parallel]
     [Method(TextDocumentNames.OnTypeFormatting, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Document/IDocumentRangeFormattingHandler.cs b/src/Protocol/Document/IDocumentRangeFormattingHandler.cs
@@ -8,7 +8,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Document
 {
-    [Serial]
+    [Parallel]
     [Method(TextDocumentNames.RangeFormatting, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Document/IPrepareRenameHandler.cs b/src/Protocol/Document/IPrepareRenameHandler.cs
@@ -8,7 +8,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Document
 {
-    [Serial]
+    [Parallel]
     [Method(TextDocumentNames.PrepareRename, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Document/IRenameHandler.cs b/src/Protocol/Document/IRenameHandler.cs
@@ -8,7 +8,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Document
 {
-    [Serial]
+    [Parallel]
     [Method(TextDocumentNames.Rename, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Document/IWillSaveWaitUntilTextDocumentHandler.cs b/src/Protocol/Document/IWillSaveWaitUntilTextDocumentHandler.cs
@@ -8,7 +8,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Document
 {
-    [Serial]
+    [Parallel]
     [Method(TextDocumentNames.WillSaveWaitUntil, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Models/WorkspaceFolder.cs b/src/Protocol/Models/WorkspaceFolder.cs
@@ -20,22 +20,22 @@ public bool Equals(WorkspaceFolder other)
         {
             if (ReferenceEquals(null, other)) return false;
             if (ReferenceEquals(this, other)) return true;
-            return Uri.Equals(other.Uri) && Name == other.Name;
+            return Equals(Uri, other.Uri) && Name == other.Name;
         }
 
         public override bool Equals(object obj)
         {
             if (ReferenceEquals(null, obj)) return false;
             if (ReferenceEquals(this, obj)) return true;
-            if (obj.GetType() != GetType()) return false;
+            if (obj.GetType() != this.GetType()) return false;
             return Equals((WorkspaceFolder) obj);
         }
 
         public override int GetHashCode()
         {
             unchecked
             {
-                return ( Uri.GetHashCode() * 397 ) ^ Name.GetHashCode();
+                return ( ( Uri != null ? Uri.GetHashCode() : 0 ) * 397 ) ^ ( Name != null ? Name.GetHashCode() : 0 );
             }
         }
 
diff --git a/src/Protocol/Window/IShowMessageRequestHandler.cs b/src/Protocol/Window/IShowMessageRequestHandler.cs
@@ -7,7 +7,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Window
 {
-    [Serial]
+    [Parallel]
     [Method(WindowNames.ShowMessageRequest, Direction.ServerToClient)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(IWindowLanguageServer), typeof(ILanguageServer))]
diff --git a/src/Protocol/Workspace/IDidChangeConfigurationHandler.cs b/src/Protocol/Workspace/IDidChangeConfigurationHandler.cs
@@ -9,7 +9,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Workspace
 {
-    [Serial]
+    [Parallel]
     [Method(WorkspaceNames.DidChangeConfiguration, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(IWorkspaceLanguageClient), typeof(ILanguageClient))]
diff --git a/src/Protocol/Workspace/IDidChangeWatchedFilesHandler.cs b/src/Protocol/Workspace/IDidChangeWatchedFilesHandler.cs
@@ -9,7 +9,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Workspace
 {
-    [Serial]
+    [Parallel]
     [Method(WorkspaceNames.DidChangeWatchedFiles, Direction.ClientToServer)]
     [GenerateHandlerMethods]
     [GenerateRequestMethods(typeof(IWorkspaceLanguageClient), typeof(ILanguageClient))]
diff --git a/test/Lsp.Tests/RequestProcessIdentifierTests.cs b/test/Lsp.Tests/RequestProcessIdentifierTests.cs
@@ -46,6 +46,12 @@ public void ShouldIdentifyAs_Default()
         [InlineData(typeof(IHoverHandler))]
         [InlineData(typeof(IReferencesHandler))]
         [InlineData(typeof(ISignatureHelpHandler))]
+        [InlineData(typeof(IDocumentFormattingHandler))]
+        [InlineData(typeof(IDocumentOnTypeFormattingHandler))]
+        [InlineData(typeof(IDocumentRangeFormattingHandler))]
+        [InlineData(typeof(IWillSaveWaitUntilTextDocumentHandler))]
+        [InlineData(typeof(IDidChangeConfigurationHandler))]
+        [InlineData(typeof(IDidChangeWatchedFilesHandler))]
         public void ShouldIdentifyAs_Parallel(Type type)
         {
             var identifier = new RequestProcessIdentifier();
@@ -61,16 +67,10 @@ public void ShouldIdentifyAs_Parallel(Type type)
         [InlineData(typeof(IDidChangeTextDocumentHandler))]
         [InlineData(typeof(IDidOpenTextDocumentHandler))]
         [InlineData(typeof(IDidSaveTextDocumentHandler))]
-        [InlineData(typeof(IDocumentFormattingHandler))]
-        [InlineData(typeof(IDocumentOnTypeFormattingHandler))]
-        [InlineData(typeof(IDocumentRangeFormattingHandler))]
-        [InlineData(typeof(IWillSaveWaitUntilTextDocumentHandler))]
         [InlineData(typeof(IExitHandler))]
         [InlineData(typeof(IShutdownHandler))]
         [InlineData(typeof(ILanguageProtocolInitializeHandler))]
         [InlineData(typeof(ILanguageProtocolInitializedHandler))]
-        [InlineData(typeof(IDidChangeConfigurationHandler))]
-        [InlineData(typeof(IDidChangeWatchedFilesHandler))]
         [InlineData(typeof(IExecuteCommandHandler))]
         public void ShouldIdentifyAs_Serial(Type type)
         {

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`
`9`	`9`	`namespace OmniSharp.Extensions.LanguageServer.Protocol.Document`
`10`	`10`	`{`
`11`		`- [Serial]`
	`11`	`+ [Parallel]`
`12`	`12`	`[Method(TextDocumentNames.DocumentFormatting, Direction.ClientToServer)]`
`13`	`13`	`[GenerateHandlerMethods]`
`14`	`14`	`[GenerateRequestMethods(typeof(ITextDocumentLanguageClient), typeof(ILanguageClient))]`
Original file line number	Diff line number	Diff line change
`@@ -20,22 +20,22 @@ public bool Equals(WorkspaceFolder other)`
`20`	`20`	`{`
`21`	`21`	`if (ReferenceEquals(null, other)) return false;`
`22`	`22`	`if (ReferenceEquals(this, other)) return true;`
`23`		`- return Uri.Equals(other.Uri) && Name == other.Name;`
	`23`	`+ return Equals(Uri, other.Uri) && Name == other.Name;`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`public override bool Equals(object obj)`
`27`	`27`	`{`
`28`	`28`	`if (ReferenceEquals(null, obj)) return false;`
`29`	`29`	`if (ReferenceEquals(this, obj)) return true;`
`30`		`- if (obj.GetType() != GetType()) return false;`
	`30`	`+ if (obj.GetType() != this.GetType()) return false;`
`31`	`31`	`return Equals((WorkspaceFolder) obj);`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`public override int GetHashCode()`
`35`	`35`	`{`
`36`	`36`	`unchecked`
`37`	`37`	`{`
`38`		`- return ( Uri.GetHashCode() * 397 ) ^ Name.GetHashCode();`
	`38`	`+ return ( ( Uri != null ? Uri.GetHashCode() : 0 ) * 397 ) ^ ( Name != null ? Name.GetHashCode() : 0 );`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`
`8`	`8`	`namespace OmniSharp.Extensions.LanguageServer.Protocol.Window`
`9`	`9`	`{`
`10`		`- [Serial]`
	`10`	`+ [Parallel]`
`11`	`11`	`[Method(WindowNames.ShowMessageRequest, Direction.ServerToClient)]`
`12`	`12`	`[GenerateHandlerMethods]`
`13`	`13`	`[GenerateRequestMethods(typeof(IWindowLanguageServer), typeof(ILanguageServer))]`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`
`10`	`10`	`namespace OmniSharp.Extensions.LanguageServer.Protocol.Workspace`
`11`	`11`	`{`
`12`		`- [Serial]`
	`12`	`+ [Parallel]`
`13`	`13`	`[Method(WorkspaceNames.DidChangeConfiguration, Direction.ClientToServer)]`
`14`	`14`	`[GenerateHandlerMethods]`
`15`	`15`	`[GenerateRequestMethods(typeof(IWorkspaceLanguageClient), typeof(ILanguageClient))]`