Removed unused private Interface property.

GammaSoul · Jul 20, 2021 · 92ba17d · 92ba17d
1 parent 8987f97
commit 92ba17d
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 24 deletions.
diff --git a/CoreRemoting.Tests/CoreRemoting.Tests.csproj b/CoreRemoting.Tests/CoreRemoting.Tests.csproj
@@ -7,14 +7,14 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Newtonsoft.Json" Version="12.0.3" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.7.1" />
+        <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.10.0" />
         <PackageReference Include="xunit" Version="2.4.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>
-        <PackageReference Include="coverlet.collector" Version="1.3.0">
+        <PackageReference Include="coverlet.collector" Version="3.0.3">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>

diff --git a/CoreRemoting.Tests/RemotingConfigurationTests.cs b/CoreRemoting.Tests/RemotingConfigurationTests.cs
@@ -1,15 +1,11 @@
 using System;
 using System.IO;
-using System.Net;
 using System.Reflection;
-using System.Threading;
-using Castle.DynamicProxy.Generators.Emitters.SimpleAST;
 using CoreRemoting.Authentication;
 using CoreRemoting.Channels.Websocket;
 using CoreRemoting.ClassicRemotingApi;
 using CoreRemoting.DependencyInjection;
 using CoreRemoting.Serialization.Binary;
-using CoreRemoting.Serialization.Bson;
 using CoreRemoting.Tests.Tools;
 using Xunit;
 

diff --git a/CoreRemoting.Tests/RpcTests.cs b/CoreRemoting.Tests/RpcTests.cs
@@ -45,7 +45,7 @@ public void Call_on_Proxy_should_be_invoked_on_remote_service()
             int serverErrorCount = 0;
 
             using var server = new RemotingServer(serverConfig);
-            server.Error += (sender, e) => serverErrorCount++;
+            server.Error += (_, _) => serverErrorCount++;
             server.Start();
 
             void ClientAction()
@@ -109,7 +109,7 @@ public void Call_on_Proxy_should_be_invoked_on_remote_service_without_MessageEnc
             int serverErrorCount = 0;
 
             using var server = new RemotingServer(serverConfig);
-            server.Error += (sender, e) => serverErrorCount++;
+            server.Error += (_, _) => serverErrorCount++;
             server.Start();
 
             void ClientAction()
@@ -165,7 +165,7 @@ public void Delegate_invoked_on_server_should_callback_client()
             int serverErrorCount = 0;
 
             using var server = new RemotingServer(serverConfig);
-            server.Error += (sender, e) => serverErrorCount++;
+            server.Error += (_, _) => serverErrorCount++;
             server.Start();
 
             void ClientAction()
@@ -218,7 +218,7 @@ public void Events_should_work_remotly()
             bool serviceEventCalled = false;
 
             using var server = new RemotingServer(serverConfig);
-            server.Error += (sender, e) => serverErrorCount++;
+            server.Error += (_, _) => serverErrorCount++;
             server.Start();
 
             using var client = new RemotingClient(
@@ -270,7 +270,7 @@ public void External_types_should_work_as_remote_service_parameters()
             int serverErrorCount = 0;
 
             using var server = new RemotingServer(serverConfig);
-            server.Error += (sender, e) => serverErrorCount++;
+            server.Error += (_, _) => serverErrorCount++;
             server.Start();
 
             void ClientAction()

diff --git a/CoreRemoting.Tests/SessionTests.cs b/CoreRemoting.Tests/SessionTests.cs
@@ -110,7 +110,7 @@ public void Client_Connect_should_throw_exception_on_invalid_auth_credentials()
             int serverErrorCount = 0;
 
             using var server = new RemotingServer(serverConfig);
-            server.Error += (sender, e) => serverErrorCount++;
+            server.Error += (_, _) => serverErrorCount++;
             server.Start();
 
             var clientAction = new Action<string, bool>((password, shouldThrow) =>

diff --git a/CoreRemoting/ServiceProxy.cs b/CoreRemoting/ServiceProxy.cs
@@ -1,5 +1,4 @@
 using System;
-using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using System.Reflection;
 using Castle.DynamicProxy;
@@ -32,9 +31,6 @@ public ServiceProxy(RemotingClient client, string serviceName = "")
                 string.IsNullOrWhiteSpace(serviceName)
                     ? serviceInterfaceType.FullName
                     : serviceName;
-
-            Interface =
-                (TServiceInterface)_client.ProxyGenerator.CreateInterfaceProxyWithoutTarget(typeof (TServiceInterface), this);
         }
 
         /// <summary>
@@ -59,13 +55,6 @@ void IServiceProxy.Shutdown()
             GC.SuppressFinalize(this);
         }
 
-        /// <summary>
-        /// Gets or sets the interface type of the proxied remote service.
-        /// </summary>
-        [SuppressMessage("ReSharper", "UnusedAutoPropertyAccessor.Local")]
-        [SuppressMessage("ReSharper", "AutoPropertyCanBeMadeGetOnly.Local")]
-        private TServiceInterface Interface { get; set; }
-
         /// <summary>
         /// Intercepts a call of a member on the proxy object. 
         /// </summary>