Fixes tests for Code Actions

Author: bjorkstromm

test/Lsp.Tests/LspRequestRouterTests.cs

@@ -94,7 +94,7 @@ public async Task ShouldRouteToCorrect_Request()
             textDocumentSyncHandler.Handle(Arg.Any<DidSaveTextDocumentParams>(), Arg.Any<CancellationToken>()).Returns(Unit.Value);
 
             var codeActionHandler = Substitute.For<ICodeActionHandler>();
-            codeActionHandler.GetRegistrationOptions().Returns(new TextDocumentRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cs") });
+            codeActionHandler.GetRegistrationOptions().Returns(new CodeActionRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cs") });
             codeActionHandler
                 .Handle(Arg.Any<CodeActionParams>(), Arg.Any<CancellationToken>())
                 .Returns(new CommandOrCodeActionContainer());
@@ -126,13 +126,13 @@ public async Task ShouldRouteToCorrect_Request_WithManyHandlers()
             textDocumentSyncHandler2.Handle(Arg.Any<DidSaveTextDocumentParams>(), Arg.Any<CancellationToken>()).Returns(Unit.Value);
 
             var codeActionHandler = Substitute.For<ICodeActionHandler>();
-            codeActionHandler.GetRegistrationOptions().Returns(new TextDocumentRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cs") });
+            codeActionHandler.GetRegistrationOptions().Returns(new CodeActionRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cs") });
             codeActionHandler
                 .Handle(Arg.Any<CodeActionParams>(), Arg.Any<CancellationToken>())
                 .Returns(new CommandOrCodeActionContainer());
 
             var codeActionHandler2 = Substitute.For<ICodeActionHandler>();
-            codeActionHandler2.GetRegistrationOptions().Returns(new TextDocumentRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cake") });
+            codeActionHandler2.GetRegistrationOptions().Returns(new CodeActionRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cake") });
             codeActionHandler2
                 .Handle(Arg.Any<CodeActionParams>(), Arg.Any<CancellationToken>())
                 .Returns(new CommandOrCodeActionContainer());

test/Lsp.Tests/MediatorTestsRequestHandlerOfTRequestTResponse.cs

@@ -46,7 +46,7 @@ public async Task RequestsCancellation()
             textDocumentSyncHandler.Handle(Arg.Any<DidSaveTextDocumentParams>(), Arg.Any<CancellationToken>()).Returns(Unit.Value);
 
             var codeActionHandler = Substitute.For<ICodeActionHandler>();
-            codeActionHandler.GetRegistrationOptions().Returns(new TextDocumentRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cs") });
+            codeActionHandler.GetRegistrationOptions().Returns(new CodeActionRegistrationOptions() { DocumentSelector = DocumentSelector.ForPattern("**/*.cs") });
             codeActionHandler
                 .Handle(Arg.Any<CodeActionParams>(), Arg.Any<CancellationToken>())
                 .Returns(async (c) => {

test/Lsp.Tests/Models/CodeActionRegistrationOptionsTests.cs

@@ -0,0 +1,45 @@
+using System;
+using FluentAssertions;
+using Newtonsoft.Json;
+using OmniSharp.Extensions.LanguageServer.Protocol;
+using OmniSharp.Extensions.LanguageServer.Protocol.Client.Capabilities;
+using OmniSharp.Extensions.LanguageServer.Protocol.Models;
+using OmniSharp.Extensions.LanguageServer.Protocol.Serialization;
+using Xunit;
+
+namespace Lsp.Tests.Models
+{
+    public class CodeActionRegistrationOptionsTests
+    {
+        [Theory, JsonFixture]
+        public void SimpleTest(string expected)
+        {
+            var model = new CodeActionRegistrationOptions() {
+                DocumentSelector = new DocumentSelector(new[] { new DocumentFilter(){
+                    Language = "csharp",
+                    Pattern = "pattern",
+                    Scheme = "scheme"
+                }, new DocumentFilter(){
+                    Language = "vb",
+                    Pattern = "pattern",
+                    Scheme = "scheme"
+                } }),
+                CodeActionKinds = new[] {
+                    CodeActionKind.QuickFix,
+                    CodeActionKind.Refactor,
+                    CodeActionKind.RefactorExtract,
+                    CodeActionKind.RefactorInline,
+                    CodeActionKind.RefactorRewrite,
+                    CodeActionKind.Source,
+                    CodeActionKind.SourceOrganizeImports
+                }
+            };
+            var result = Fixture.SerializeObject(model);
+
+            result.Should().Be(expected);
+
+            var deresult = new Serializer(ClientVersion.Lsp3).DeserializeObject<CodeActionRegistrationOptions>(expected);
+            deresult.Should().BeEquivalentTo(model);
+        }
+    }
+}

test/Lsp.Tests/Models/CodeActionRegistrationOptionsTests_$SimpleTest.json

@@ -0,0 +1,23 @@
+{
+    "codeActionKinds": [
+        "quickfix",
+        "refactor",
+        "refactor.extract",
+        "refactor.inline",
+        "refactor.rewrite",
+        "source",
+        "source.organizeImports"
+    ],
+    "documentSelector": [
+        {
+            "language": "csharp",
+            "scheme": "scheme",
+            "pattern": "pattern"
+        },
+        {
+            "language": "vb",
+            "scheme": "scheme",
+            "pattern": "pattern"
+        }
+    ]
+}