Removed the constructor overloads of MockFileVersionInfo and set the default value for the optional parameters

Author: t4m45

src/TestableIO.System.IO.Abstractions.TestingHelpers/MockFileVersionInfo.cs

@@ -13,24 +13,24 @@ public class MockFileVersionInfo : FileVersionInfoBase
         /// <inheritdoc />
         public MockFileVersionInfo(
             string fileName,
-            string fileVersion,
-            string productVersion,
-            string fileDescription,
-            string productName,
-            string companyName,
-            string comments,
-            string internalName,
-            bool isDebug,
-            bool isPatched,
-            bool isPrivateBuild,
-            bool isPreRelease,
-            bool isSpecialBuild,
-            string language,
-            string legalCopyright,
-            string legalTrademarks,
-            string originalFilename,
-            string privateBuild,
-            string specialBuild)
+            string fileVersion = null,
+            string productVersion = null,
+            string fileDescription = null,
+            string productName = null,
+            string companyName = null,
+            string comments = null,
+            string internalName = null,
+            bool isDebug = false,
+            bool isPatched = false,
+            bool isPrivateBuild = false,
+            bool isPreRelease = false,
+            bool isSpecialBuild = false,
+            string language = null,
+            string legalCopyright = null,
+            string legalTrademarks = null,
+            string originalFilename = null,
+            string privateBuild = null,
+            string specialBuild = null)
         {
             FileName = fileName;
             FileVersion = fileVersion;
@@ -73,18 +73,6 @@ public MockFileVersionInfo(
             ProductPrivatePart = productPrivate;
         }
 
-        /// <inheritdoc/>
-        public MockFileVersionInfo(string fileName)
-            : this(fileName, null, null, null, null, null, null, null, false, false, false, false, false, null, null, null, null, null, null) { }
-
-        /// <inheritdoc/>
-        public MockFileVersionInfo(string fileName, string fileVersion, string productVersion)
-            : this(fileName, fileVersion, productVersion, null, null, null, null, null, false, false, false, false, false, null, null, null, null, null, null) { }
-
-        /// <inheritdoc/>
-        public MockFileVersionInfo()
-            : this(null, null, null, null, null, null, null, null, false, false, false, false, false, null, null, null, null, null, null) { }
-
         /// <inheritdoc/>
         public override string FileName { get; }
 

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileVersionInfoFactoryTests.cs

@@ -14,7 +14,7 @@ public class MockFileVersionInfoFactoryTests
         public void MockFileVersionInfoFactory_GetVersionInfo_ShouldReturnTheFileVersionInfoOfTheMockFileData()
         {
             // Arrange
-            var fileVersionInfo = new MockFileVersionInfo();
+            var fileVersionInfo = new MockFileVersionInfo(@"c:\a.txt");
             var fileSystem = new MockFileSystem(new Dictionary<string, MockFileData>
             {
                 { @"c:\a.txt", new MockFileData("Demo text content") { FileVersionInfo = fileVersionInfo } }

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileVersionInfoTests.cs

@@ -71,7 +71,7 @@ public void MockFileVersionInfo_Constructor_ShouldSetFileAndProductVersionNumber
         public void MockFileVersionInfo_Constructor_ShouldNotSetFileAndProductVersionNumbersIfFileAndProductVersionAreNull()
         {
             // Act
-            var mockFileVersionInfo = new MockFileVersionInfo();
+            var mockFileVersionInfo = new MockFileVersionInfo(@"c:\a.txt");
 
             // Assert
             Assert.That(mockFileVersionInfo.FileMajorPart, Is.EqualTo(0));