Update test snapshots

JoeRobich · JoeRobich · commit 53a8f0847f12 · 2023-09-13T23:22:25.000-07:00
diff --git a/test/Dap.Tests/FoundationTests.cs b/test/Dap.Tests/FoundationTests.cs
@@ -140,7 +140,7 @@ public void HandlersShouldMatchParamsMethodAttribute(Type type)
             lhs.Direction.Should().Be(rhs.Direction, $"{type.FullName} direction does not match {paramsType.FullName}");
         }
 
-        [Theory(DisplayName = "Handler interfaces should have a abstract class")]
+        [Theory(DisplayName = "Handler interfaces should have an abstract class")]
         [ClassData(typeof(TypeHandlerData))]
         public void HandlersShouldAbstractClass(IHandlerTypeDescriptor descriptor)
         {
@@ -226,17 +226,19 @@ public void HandlersShouldHaveExpectedExtensionMethodsBasedOnDirection(
                                                      .Distinct()
                                                      .ToHashSet();
 
-            var expectedEventRegistries = descriptor.Direction switch {
-                Direction.ClientToServer => new (string type, Func<ParameterInfo, bool> matcher)[] { ( "Server", info => info.ParameterType.Name.EndsWith("ServerRegistry") ) },
-                Direction.ServerToClient => new (string type, Func<ParameterInfo, bool> matcher)[] { ( "Client", info => info.ParameterType.Name.EndsWith("ClientRegistry") ) },
+            var expectedEventRegistries = descriptor.Direction switch
+            {
+                Direction.ClientToServer => new (string type, Func<ParameterInfo, bool> matcher)[] { ("Server", info => info.ParameterType.Name.EndsWith("ServerRegistry")) },
+                Direction.ServerToClient => new (string type, Func<ParameterInfo, bool> matcher)[] { ("Client", info => info.ParameterType.Name.EndsWith("ClientRegistry")) },
                 Direction.Bidirectional => new (string type, Func<ParameterInfo, bool> matcher)[]
                     { ( "Server", info => info.ParameterType.Name.EndsWith("ServerRegistry") ), ( "Client", info => info.ParameterType.Name.EndsWith("ClientRegistry") ) },
                 _ => throw new NotImplementedException(descriptor.HandlerType.FullName)
             };
 
-            var expectedRequestHandlers = descriptor.Direction switch {
-                Direction.ClientToServer => new (string type, Func<ParameterInfo, bool> matcher)[] { ( "Server", info => info.ParameterType.Name.EndsWith("Client") ) },
-                Direction.ServerToClient => new (string type, Func<ParameterInfo, bool> matcher)[] { ( "Client", info => info.ParameterType.Name.EndsWith("Server") ) },
+            var expectedRequestHandlers = descriptor.Direction switch
+            {
+                Direction.ClientToServer => new (string type, Func<ParameterInfo, bool> matcher)[] { ("Server", info => info.ParameterType.Name.EndsWith("Client")) },
+                Direction.ServerToClient => new (string type, Func<ParameterInfo, bool> matcher)[] { ("Client", info => info.ParameterType.Name.EndsWith("Server")) },
                 Direction.Bidirectional => new (string type, Func<ParameterInfo, bool> matcher)[]
                     { ( "Server", info => info.ParameterType.Name.EndsWith("Client") ), ( "Client", info => info.ParameterType.Name.EndsWith("Server") ) },
                 _ => throw new NotImplementedException(descriptor.HandlerType.FullName)
@@ -526,7 +528,7 @@ private static Type GetExtensionClass(IHandlerTypeDescriptor descriptor)
         {
             var name = GetExtensionClassName(descriptor);
             return descriptor.HandlerType.Assembly.GetExportedTypes()
-                             .FirstOrDefault(z => z.IsClass && z.IsAbstract && ( z.Name == name || z.Name == name + "Base" ));
+                             .FirstOrDefault(z => z.IsClass && z.IsAbstract && (z.Name == name || z.Name == name + "Base"));
         }
 
         private static string GetOnMethodName(IHandlerTypeDescriptor descriptor) => "On" + SpecialCasedHandlerName(descriptor);
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.00AssemblyCapabilityKeys.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.00AssemblyCapabilityKeys.verified.cs
@@ -1,5 +1,6 @@
 ﻿//HintName: AssemblyCapabilityKeys.cs
 using MediatR;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.JsonRpc;
 using OmniSharp.Extensions.JsonRpc.Generation;
 using OmniSharp.Extensions.LanguageServer.Protocol;
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.01Test0_SymbolInformationParams.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.01Test0_SymbolInformationParams.verified.cs
@@ -2,6 +2,7 @@
 #nullable enable
 using System.Diagnostics;
 using MediatR;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.JsonRpc;
 using OmniSharp.Extensions.JsonRpc.Generation;
 using OmniSharp.Extensions.LanguageServer.Protocol.Client;
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.02Test0_WorkspaceSymbolParams.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.02Test0_WorkspaceSymbolParams.verified.cs
@@ -2,6 +2,7 @@
 #nullable enable
 using System.Diagnostics;
 using MediatR;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.JsonRpc;
 using OmniSharp.Extensions.JsonRpc.Generation;
 using OmniSharp.Extensions.LanguageServer.Protocol.Client;
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.04SymbolInformationParams.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.04SymbolInformationParams.verified.cs
@@ -1,6 +1,7 @@
 ﻿//HintName: SymbolInformationParams.cs
 using MediatR;
 using Microsoft.Extensions.DependencyInjection;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.DebugAdapter.Protocol;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Events;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Models;
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.05WorkspaceSymbol.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.05WorkspaceSymbol.verified.cs
@@ -1,6 +1,7 @@
 ﻿//HintName: WorkspaceSymbol.cs
 using MediatR;
 using Microsoft.Extensions.DependencyInjection;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.DebugAdapter.Protocol;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Events;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Models;
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.06WorkspaceSymbolParams.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.06WorkspaceSymbolParams.verified.cs
@@ -1,6 +1,7 @@
 ﻿//HintName: WorkspaceSymbolParams.cs
 using MediatR;
 using Microsoft.Extensions.DependencyInjection;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.DebugAdapter.Protocol;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Events;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Models;
diff --git a/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.08WorkspaceSymbolRegistrationOptions.verified.cs b/test/Generation.Tests/snapshots/LspFeatureTests.Supports_Params_Type_As_Source.08WorkspaceSymbolRegistrationOptions.verified.cs
@@ -1,6 +1,7 @@
 ﻿//HintName: WorkspaceSymbolRegistrationOptions.cs
 using System.Diagnostics;
 using MediatR;
+using Newtonsoft.Json.Linq;
 using OmniSharp.Extensions.JsonRpc;
 using OmniSharp.Extensions.JsonRpc.Generation;
 using OmniSharp.Extensions.LanguageServer.Protocol.Client;
diff --git a/test/Lsp.Tests/FoundationTests.cs b/test/Lsp.Tests/FoundationTests.cs
@@ -142,7 +142,7 @@ public void Debugger_Display_Should_Not_Throw(Type type)
         {
             var instance = Activator.CreateInstance(type);
             var property = type.GetProperty("DebuggerDisplay", BindingFlags.NonPublic | BindingFlags.Instance)!;
-            var a1 = () => ( property.GetValue(instance) as string )!;
+            var a1 = () => (property.GetValue(instance) as string)!;
             var a2 = () => instance!.ToString()!;
 
             a1.Should().NotThrow().And.NotBeNull();
@@ -205,7 +205,7 @@ public void Registration_Converters_Should_Have_THe_Same_Properties(Type type)
             source.Should().Contain(destination);
         }
 
-        [Theory(DisplayName = "Handler interfaces should have a abstract class")]
+        [Theory(DisplayName = "Handler interfaces should have an class")]
         [ClassData(typeof(TypeHandlerData))]
         public void HandlersShouldAbstractClass(ILspHandlerTypeDescriptor descriptor)
         {
@@ -392,7 +392,7 @@ Func<MethodInfo, bool> ForAnyParameter(Func<ParameterInfo, bool> m)
                 var returns = ForAnyParameter(info => info.ParameterType.GetGenericArguments().LastOrDefault() == returnType);
                 var isAction = ForAnyParameter(info => info.ParameterType.Name.StartsWith(nameof(Action)));
                 var isFunc = ForAnyParameter(info => info.ParameterType.Name.StartsWith("Func"));
-                
+
                 var takesParameter = ForAnyParameter(info => info.ParameterType.GetGenericArguments().FirstOrDefault() == descriptor.ParamsType);
                 var takesCapability = ForAnyParameter(info => info.ParameterType.GetGenericArguments().Skip(1).FirstOrDefault() == descriptor.CapabilityType);
 
@@ -772,7 +772,7 @@ public TypeHandlerExtensionData()
                 foreach (var type in typeof(CompletionParams).Assembly.ExportedTypes
                                                              .Where(z => z.IsInterface && typeof(IJsonRpcHandler).IsAssignableFrom(z))
                                                              .Where(z => !z.Name.EndsWith("Manager"))
-                                                             .Except(new[] { typeof(ITextDocumentSyncHandler) , typeof(INotebookDocumentSyncHandler)})
+                                                             .Except(new[] { typeof(ITextDocumentSyncHandler), typeof(INotebookDocumentSyncHandler) })
                         )
                 {
                     if (type.IsGenericTypeDefinition && !MethodAttribute.AllFrom(type).Any()) continue;
@@ -818,7 +818,7 @@ private static Type GetExtensionClass(IHandlerTypeDescriptor descriptor)
         {
             var name = GetExtensionClassName(descriptor);
             return descriptor.HandlerType.Assembly.GetExportedTypes()
-                             .FirstOrDefault(z => z.IsClass && z.IsAbstract && ( z.Name == name || z.Name == name + "Base" ))!;
+                             .FirstOrDefault(z => z.IsClass && z.IsAbstract && (z.Name == name || z.Name == name + "Base"))!;
         }
 
         private static string GetOnMethodName(IHandlerTypeDescriptor descriptor)

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ public void Debugger_Display_Should_Not_Throw(Type type)`
`142`	`142`	`{`
`143`	`143`	`var instance = Activator.CreateInstance(type);`
`144`	`144`	`var property = type.GetProperty("DebuggerDisplay", BindingFlags.NonPublic \| BindingFlags.Instance)!;`
`145`		`- var a1 = () => ( property.GetValue(instance) as string )!;`
	`145`	`+ var a1 = () => (property.GetValue(instance) as string)!;`
`146`	`146`	`var a2 = () => instance!.ToString()!;`
`147`	`147`
`148`	`148`	`a1.Should().NotThrow().And.NotBeNull();`
`@@ -205,7 +205,7 @@ public void Registration_Converters_Should_Have_THe_Same_Properties(Type type)`
`205`	`205`	`source.Should().Contain(destination);`
`206`	`206`	`}`
`207`	`207`
`208`		`- [Theory(DisplayName = "Handler interfaces should have a abstract class")]`
	`208`	`+ [Theory(DisplayName = "Handler interfaces should have an class")]`
`209`	`209`	`[ClassData(typeof(TypeHandlerData))]`
`210`	`210`	`public void HandlersShouldAbstractClass(ILspHandlerTypeDescriptor descriptor)`
`211`	`211`	`{`
`@@ -392,7 +392,7 @@ Func<MethodInfo, bool> ForAnyParameter(Func<ParameterInfo, bool> m)`
`392`	`392`	`var returns = ForAnyParameter(info => info.ParameterType.GetGenericArguments().LastOrDefault() == returnType);`
`393`	`393`	`var isAction = ForAnyParameter(info => info.ParameterType.Name.StartsWith(nameof(Action)));`
`394`	`394`	`var isFunc = ForAnyParameter(info => info.ParameterType.Name.StartsWith("Func"));`
`395`		`-`
	`395`	`+`
`396`	`396`	`var takesParameter = ForAnyParameter(info => info.ParameterType.GetGenericArguments().FirstOrDefault() == descriptor.ParamsType);`
`397`	`397`	`var takesCapability = ForAnyParameter(info => info.ParameterType.GetGenericArguments().Skip(1).FirstOrDefault() == descriptor.CapabilityType);`
`398`	`398`
`@@ -772,7 +772,7 @@ public TypeHandlerExtensionData()`
`772`	`772`	`foreach (var type in typeof(CompletionParams).Assembly.ExportedTypes`
`773`	`773`	`.Where(z => z.IsInterface && typeof(IJsonRpcHandler).IsAssignableFrom(z))`
`774`	`774`	`.Where(z => !z.Name.EndsWith("Manager"))`
`775`		`- .Except(new[] { typeof(ITextDocumentSyncHandler) , typeof(INotebookDocumentSyncHandler)})`
	`775`	`+ .Except(new[] { typeof(ITextDocumentSyncHandler), typeof(INotebookDocumentSyncHandler) })`
`776`	`776`	`)`
`777`	`777`	`{`
`778`	`778`	`if (type.IsGenericTypeDefinition && !MethodAttribute.AllFrom(type).Any()) continue;`
`@@ -818,7 +818,7 @@ private static Type GetExtensionClass(IHandlerTypeDescriptor descriptor)`
`818`	`818`	`{`
`819`	`819`	`var name = GetExtensionClassName(descriptor);`
`820`	`820`	`return descriptor.HandlerType.Assembly.GetExportedTypes()`
`821`		`- .FirstOrDefault(z => z.IsClass && z.IsAbstract && ( z.Name == name \|\| z.Name == name + "Base" ))!;`
	`821`	`+ .FirstOrDefault(z => z.IsClass && z.IsAbstract && (z.Name == name \|\| z.Name == name + "Base"))!;`
`822`	`822`	`}`
`823`	`823`
`824`	`824`	`private static string GetOnMethodName(IHandlerTypeDescriptor descriptor)`