Fix some (lots of) issues reported by analyzers. (#1125)

Fix some (lots of) issues reported by analyzers.

Author: drieseng

.editorconfig_soon

@@ -0,0 +1,48 @@
+<Project>
+    <Import Project="$(MSBuildThisFileFullPath).user" Condition="Exists('$(MSBuildThisFileFullPath).user')" />
+    
+    <!--
+        Assembly Info properties that apply to all projects/assemblies.
+    -->
+    <PropertyGroup>
+        <SignAssembly>true</SignAssembly>
+        <AssemblyOriginatorKeyFile>$(MSBuildThisFileDirectory)src\Renci.SshNet.snk</AssemblyOriginatorKeyFile>
+        <GenerateDocumentationFile>true</GenerateDocumentationFile>
+        <LangVersion>latest</LangVersion>
+        <!--
+        <WarningLevel>9999</WarningLevel>
+        <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+        -->
+        <AutoGenerateBindingRedirects>false</AutoGenerateBindingRedirects>
+    </PropertyGroup>
+
+    <!--
+        Code analysis properties.
+    -->
+    <PropertyGroup>
+        <EnableNETAnalyzers>false</EnableNETAnalyzers>
+        <AnalysisLevel>preview-All</AnalysisLevel>
+        <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
+    </PropertyGroup>
+
+    <!--
+        Add the stylecop config to each project.
+    -->
+    <ItemGroup>
+        <AdditionalFiles Include="..\..\stylecop.json" Link="stylecop.json" />
+    </ItemGroup>
+
+    <!--
+        Use fixed version of analyzers.
+    -->
+    <ItemGroup>
+        <!--
+        <PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" Version="8.0.0-preview1.23165.1" PrivateAssets="all" />
+        <PackageReference Include="StyleCop.Analyzers" Version="1.2.0-beta.435" PrivateAssets="all" />
+        <PackageReference Include="Meziantou.Analyzer" Version="2.0.52" PrivateAssets="all" />
+        -->
+        <!--
+        <PackageReference Include="SonarAnalyzer.CSharp" Version="8.55.0.65544" PrivateAssets="all" />
+        -->
+    </ItemGroup>
+</Project>

Directory.Build.props

@@ -1,26 +1,37 @@
-using Microsoft.VisualStudio.TestTools.UnitTesting;
+using System;
+
+using Microsoft.VisualStudio.TestTools.UnitTesting;
 using Renci.SshNet.Common;
 using Renci.SshNet.Tests.Common;
 
 namespace Renci.SshNet.Tests.Classes.Common
-{   
-    /// <summary>
-    ///This is a test class for ObjectIdentifierTest and is intended
-    ///to contain all ObjectIdentifierTest Unit Tests
-    ///</summary>
+{
     [TestClass]
-    [Ignore] // placeholder for actual test
     public class ObjectIdentifierTest : TestBase
     {
-        /// <summary>
-        ///A test for ObjectIdentifier Constructor
-        ///</summary>
         [TestMethod]
-        public void ObjectIdentifierConstructorTest()
+        public void Constructor_IdentifiersIsNull()
         {
-            ulong[] identifiers = null; // TODO: Initialize to an appropriate value
-            ObjectIdentifier target = new ObjectIdentifier(identifiers);
-            Assert.Inconclusive("TODO: Implement code to verify target");
+            const ulong[] identifiers = null;
+
+            var actualException = Assert.ThrowsException<ArgumentNullException>(() => new ObjectIdentifier(identifiers));
+
+            Assert.AreEqual(typeof(ArgumentNullException), actualException.GetType());
+            Assert.IsNull(actualException.InnerException);
+            Assert.AreEqual(nameof(identifiers), actualException.ParamName);
+        }
+
+        [TestMethod]
+        public void Constructor_LengthOfIdentifiersIsLessThanTwo()
+        {
+            var identifiers = new[] { 5UL };
+
+            var actualException = Assert.ThrowsException<ArgumentException>(() => new ObjectIdentifier(identifiers));
+
+            Assert.AreEqual(typeof(ArgumentException), actualException.GetType());
+            Assert.IsNull(actualException.InnerException);
+            ArgumentExceptionAssert.MessageEquals("Must contain at least two elements.", actualException);
+            Assert.AreEqual(nameof(identifiers), actualException.ParamName);
         }
     }
 }

README.md

@@ -1,13 +1,15 @@
-using Microsoft.VisualStudio.TestTools.UnitTesting;
-using Renci.SshNet.Common;
-using Renci.SshNet.Connection;
-using Renci.SshNet.Tests.Common;
-using System;
+using System;
 using System.Collections.Generic;
 using System.Net;
 using System.Net.Sockets;
 using System.Threading;
 
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+using Renci.SshNet.Common;
+using Renci.SshNet.Connection;
+using Renci.SshNet.Tests.Common;
+
 namespace Renci.SshNet.Tests.Classes.Connection
 {
     [TestClass]

src/Renci.SshNet.Tests/Classes/Common/ObjectIdentifierTest.cs

@@ -6,7 +6,6 @@ namespace Renci.SshNet.Tests.Classes
     /// <summary>
     /// Provides data for message events.
     /// </summary>
-    /// <typeparam name="T">Message type</typeparam>
     [TestClass]
     public class MessageEventArgsTest : TestBase
     {

src/Renci.SshNet.Tests/Classes/Connection/ProtocolVersionExchangeTest_ConnectionClosedByServer_NoDataSentByServer.cs

@@ -8,7 +8,6 @@ namespace Renci.SshNet.Tests.Classes.Security.Cryptography
     /// <summary>
     /// Provides HMAC algorithm implementation.
     /// </summary>
-    /// <typeparam name="T"></typeparam>
     [TestClass]
     public class HMacTest : TestBase
     {

src/Renci.SshNet.Tests/Classes/MessageEventArgsTest.cs

@@ -147,17 +147,17 @@ protected override void LoadData()
             base.LoadData();
 
             Information = new SftpFileSytemInformation(ReadUInt64(), // FileSystemBlockSize
-                                                        ReadUInt64(), // BlockSize
-                                                        ReadUInt64(), // TotalBlocks
-                                                        ReadUInt64(), // FreeBlocks
-                                                        ReadUInt64(), // AvailableBlocks
-                                                        ReadUInt64(), // TotalNodes
-                                                        ReadUInt64(), // FreeNodes
-                                                        ReadUInt64(), // AvailableNodes
-                                                        ReadUInt64(), // Sid
-                                                        ReadUInt64(), // Flags
-                                                        ReadUInt64()  // MaxNameLenght
-                                                        );
+                                                       ReadUInt64(), // BlockSize
+                                                       ReadUInt64(), // TotalBlocks
+                                                       ReadUInt64(), // FreeBlocks
+                                                       ReadUInt64(), // AvailableBlocks
+                                                       ReadUInt64(), // TotalNodes
+                                                       ReadUInt64(), // FreeNodes
+                                                       ReadUInt64(), // AvailableNodes
+                                                       ReadUInt64(), // Sid
+                                                       ReadUInt64(), // Flags
+                                                       ReadUInt64()  // MaxNameLenght
+                                                       );
         }
 
         protected override void SaveData()

src/Renci.SshNet.Tests/Classes/Security/Cryptography/HMacTest.cs

@@ -40,7 +40,7 @@ public AsyncSocketListener(IPEndPoint endPoint)
         /// <value>
         /// <see langword="true"/> to invoke <see cref="Socket.Shutdown(SocketShutdown)"/> on the <see cref="Socket"/> that is used
         /// to handle the communication with the remote host, when the remote host has closed the connection; otherwise,
-        /// <see langword="false""/>. The default is <see langword="true"/>.
+        /// <see langword="false"/>. The default is <see langword="true"/>.
         /// </value>
         public bool ShutdownRemoteCommunicationSocket { get; set; }
 

src/Renci.SshNet.Tests/Classes/Sftp/SftpStatVfsResponseBuilder.cs

@@ -1,9 +1,16 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-	<LangVersion>7.3</LangVersion>
-	<SignAssembly>true</SignAssembly>
-    <AssemblyOriginatorKeyFile>..\Renci.SshNet.snk</AssemblyOriginatorKeyFile>
-	<TargetFrameworks>net462;net6.0;net7.0</TargetFrameworks>
+    <TargetFrameworks>net462;net6.0;net7.0</TargetFrameworks>
+    <!--
+          Even though we're not interested in producing XML docs for test projects, we have to enable this in order to have the .NET Compiler
+          Platform analyzers produce the IDE0005 (Remove unnecessary import) diagnostic.
+            
+          To avoid warnings for missing XML docs, we add CS1591 (Missing XML comment for publicly visible type or member) to the NoWarn property.
+
+          We can stop producing XML docs for test projects (and remove the NoWarn for CS1591) once the following issue is fixed:
+          https://github.com/dotnet/roslyn/issues/41640.
+      -->
+    <NoWarn>$(NoWarn);CS1591</NoWarn>
   </PropertyGroup>
 
   <PropertyGroup>