Avoid InternalsVisibleTo

vbreuss · vbreuss · commit 268ac973245f · 2025-02-22T20:24:59.000+01:00
diff --git a/src/TestableIO.System.IO.Abstractions.TestingHelpers/TestableIO.System.IO.Abstractions.TestingHelpers.csproj b/src/TestableIO.System.IO.Abstractions.TestingHelpers/TestableIO.System.IO.Abstractions.TestingHelpers.csproj
@@ -11,11 +11,4 @@
         <ProjectReference Include="../TestableIO.System.IO.Abstractions.Wrappers/TestableIO.System.IO.Abstractions.Wrappers.csproj"/>
     </ItemGroup>
 
-    <ItemGroup Condition="'$(Configuration)' != 'Release'">
-        <InternalsVisibleTo Include="System.IO.Abstractions.TestingHelpers.Tests"/>
-    </ItemGroup>
-    <ItemGroup Condition="'$(Configuration)' == 'Release'">
-        <InternalsVisibleTo Include="System.IO.Abstractions.TestingHelpers.Tests" PublicKey="002400000480000094000000060200000024000052534131000400000100010051bf2aa00ba30d507d4cebcab1751dfa13768a6f5235ce52da572260e33a11f52b87707f858fe4bbe32cd51830a8dd73245f688902707fa797c07205ff9b5212f93760d52f6d13022a286ff7daa13a0cd9eb958e888fcd7d9ed1f7cf76b19a5391835a7b633418a5f584d10925d76810f782f6b814cc34a2326b438abdc3b5bd"/>
-    </ItemGroup>
-
 </Project>
diff --git a/src/TestableIO.System.IO.Abstractions.Wrappers/TestableIO.System.IO.Abstractions.Wrappers.csproj b/src/TestableIO.System.IO.Abstractions.Wrappers/TestableIO.System.IO.Abstractions.Wrappers.csproj
@@ -18,14 +18,5 @@
             <_Parameter1>true</_Parameter1>
         </AssemblyAttribute>
     </ItemGroup>
-    
-    <ItemGroup Condition="'$(Configuration)' != 'Release'">
-        <InternalsVisibleTo Include="System.IO.Abstractions.Tests"/>
-        <InternalsVisibleTo Include="DynamicProxyGenAssembly2"/>
-    </ItemGroup>
-    <ItemGroup Condition="'$(Configuration)' == 'Release'">
-        <InternalsVisibleTo Include="System.IO.Abstractions.Tests" PublicKey="$(PublicKey)"/>
-        <InternalsVisibleTo Include="DynamicProxyGenAssembly2" PublicKey="0024000004800000940000000602000000240000525341310004000001000100c547cac37abd99c8db225ef2f6c8a3602f3b3606cc9891605d02baa56104f4cfc0734aa39b93bf7852f7d9266654753cc297e7d2edfe0bac1cdcf9f717241550e0a7b191195b7667bb4f64bcb8e2121380fd1d9d46ad2d92d2d15605093924cceaf74c4861eff62abf69b9291ed0a340e113be11e6a7d3113e92484cf7045cc7"/>
-    </ItemGroup>
 
 </Project>
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net472.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net472.txt
@@ -1,5 +1,4 @@
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETFramework,Version=v4.7.2", FrameworkDisplayName=".NET Framework 4.7.2")]
 namespace System.IO.Abstractions.TestingHelpers
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net6.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net6.0.txt
@@ -1,5 +1,4 @@
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v6.0", FrameworkDisplayName=".NET 6.0")]
 namespace System.IO.Abstractions.TestingHelpers
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net8.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net8.0.txt
@@ -1,5 +1,4 @@
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v8.0", FrameworkDisplayName=".NET 8.0")]
 namespace System.IO.Abstractions.TestingHelpers
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net9.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_net9.0.txt
@@ -1,5 +1,4 @@
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v9.0", FrameworkDisplayName=".NET 9.0")]
 namespace System.IO.Abstractions.TestingHelpers
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_netstandard2.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_netstandard2.0.txt
@@ -1,5 +1,4 @@
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETStandard,Version=v2.0", FrameworkDisplayName=".NET Standard 2.0")]
 namespace System.IO.Abstractions.TestingHelpers
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_netstandard2.1.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.TestingHelpers_netstandard2.1.txt
@@ -1,5 +1,4 @@
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETStandard,Version=v2.1", FrameworkDisplayName=".NET Standard 2.1")]
 namespace System.IO.Abstractions.TestingHelpers
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net472.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net472.txt
@@ -1,7 +1,5 @@
 [assembly: System.CLSCompliant(true)]
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETFramework,Version=v4.7.2", FrameworkDisplayName=".NET Framework 4.7.2")]
 namespace System.IO.Abstractions
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net6.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net6.0.txt
@@ -1,7 +1,5 @@
 [assembly: System.CLSCompliant(true)]
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v6.0", FrameworkDisplayName=".NET 6.0")]
 namespace System.IO.Abstractions
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net8.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net8.0.txt
@@ -1,7 +1,5 @@
 [assembly: System.CLSCompliant(true)]
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v8.0", FrameworkDisplayName=".NET 8.0")]
 namespace System.IO.Abstractions
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net9.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_net9.0.txt
@@ -1,7 +1,5 @@
 [assembly: System.CLSCompliant(true)]
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v9.0", FrameworkDisplayName=".NET 9.0")]
 namespace System.IO.Abstractions
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_netstandard2.0.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_netstandard2.0.txt
@@ -1,7 +1,5 @@
 [assembly: System.CLSCompliant(true)]
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETStandard,Version=v2.0", FrameworkDisplayName=".NET Standard 2.0")]
 namespace System.IO.Abstractions
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_netstandard2.1.txt b/tests/TestableIO.System.IO.Abstractions.Api.Tests/Expected/TestableIO.System.IO.Abstractions.Wrappers_netstandard2.1.txt
@@ -1,7 +1,5 @@
 [assembly: System.CLSCompliant(true)]
 [assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]
-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]
 [assembly: System.Runtime.Versioning.TargetFramework(".NETStandard,Version=v2.1", FrameworkDisplayName=".NET Standard 2.1")]
 namespace System.IO.Abstractions
 {
diff --git a/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/Polyfills/SupportedOSPlatformAttribute.cs b/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/Polyfills/SupportedOSPlatformAttribute.cs
@@ -0,0 +1,12 @@
+#if !FEATURE_SUPPORTED_OS_ATTRIBUTE
+namespace System.Runtime.Versioning
+{
+    [AttributeUsage(AttributeTargets.All)]
+    internal class SupportedOSPlatformAttribute : Attribute
+    {
+        public SupportedOSPlatformAttribute(string _)
+        {
+        }
+    }
+}
+#endif
diff --git a/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/ProductVersionParserTests.cs b/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/ProductVersionParserTests.cs
@@ -12,15 +12,15 @@ public void ProductVersionParser_Parse_ShouldIgnoreTheSegmentsWhenThereAreMoreTh
             string productVersion = "1.2.3.4.5";
 
             // Act
-            var parsedProductVersion = ProductVersionParser.Parse(productVersion);
+            var versionInfo = new MockFileVersionInfo("foo", productVersion: productVersion);
 
             // Assert
             Assert.Multiple(() =>
             {
-                Assert.That(parsedProductVersion.Major, Is.Zero);
-                Assert.That(parsedProductVersion.Minor, Is.Zero);
-                Assert.That(parsedProductVersion.Build, Is.Zero);
-                Assert.That(parsedProductVersion.PrivatePart, Is.Zero);
+                Assert.That(versionInfo.ProductMajorPart, Is.Zero);
+                Assert.That(versionInfo.ProductMinorPart, Is.Zero);
+                Assert.That(versionInfo.ProductBuildPart, Is.Zero);
+                Assert.That(versionInfo.ProductPrivatePart, Is.Zero);
             });
         }
 
@@ -37,15 +37,15 @@ public void ProductVersionParser_Parse_ShouldSkipTheRestOfTheSegmentsWhenOneIsNo
             int expectedRevision)
         {
             // Act
-            var parsedProductVersion = ProductVersionParser.Parse(productVersion);
+            var versionInfo = new MockFileVersionInfo("foo", productVersion: productVersion);
 
             // Assert
             Assert.Multiple(() =>
             {
-                Assert.That(parsedProductVersion.Major, Is.EqualTo(expectedMajor));
-                Assert.That(parsedProductVersion.Minor, Is.EqualTo(expectedMinor));
-                Assert.That(parsedProductVersion.Build, Is.EqualTo(expectedBuild));
-                Assert.That(parsedProductVersion.PrivatePart, Is.EqualTo(expectedRevision));
+                Assert.That(versionInfo.ProductMajorPart, Is.EqualTo(expectedMajor));
+                Assert.That(versionInfo.ProductMinorPart, Is.EqualTo(expectedMinor));
+                Assert.That(versionInfo.ProductBuildPart, Is.EqualTo(expectedBuild));
+                Assert.That(versionInfo.ProductPrivatePart, Is.EqualTo(expectedRevision));
             });
         }
 
@@ -66,15 +66,15 @@ public void ProductVersionParser_Parse_ShouldSkipTheRestOfTheSegmentsWhenOneCont
             int expectedRevision)
         {
             // Act
-            var parsedProductVersion = ProductVersionParser.Parse(productVersion);
+            var versionInfo = new MockFileVersionInfo("foo", productVersion: productVersion);
 
             // Assert
             Assert.Multiple(() =>
             {
-                Assert.That(parsedProductVersion.Major, Is.EqualTo(expectedMajor));
-                Assert.That(parsedProductVersion.Minor, Is.EqualTo(expectedMinor));
-                Assert.That(parsedProductVersion.Build, Is.EqualTo(expectedBuild));
-                Assert.That(parsedProductVersion.PrivatePart, Is.EqualTo(expectedRevision));
+                Assert.That(versionInfo.ProductMajorPart, Is.EqualTo(expectedMajor));
+                Assert.That(versionInfo.ProductMinorPart, Is.EqualTo(expectedMinor));
+                Assert.That(versionInfo.ProductBuildPart, Is.EqualTo(expectedBuild));
+                Assert.That(versionInfo.ProductPrivatePart, Is.EqualTo(expectedRevision));
             });
         }
 
@@ -92,15 +92,15 @@ public void ProductVersionParser_Parse_ShouldParseEachProvidedSegment(
             int expectedRevision)
         {
             // Act
-            var parsedProductVersion = ProductVersionParser.Parse(productVersion);
+            var versionInfo = new MockFileVersionInfo("foo", productVersion: productVersion);
 
             // Assert
             Assert.Multiple(() =>
             {
-                Assert.That(parsedProductVersion.Major, Is.EqualTo(expectedMajor));
-                Assert.That(parsedProductVersion.Minor, Is.EqualTo(expectedMinor));
-                Assert.That(parsedProductVersion.Build, Is.EqualTo(expectedBuild));
-                Assert.That(parsedProductVersion.PrivatePart, Is.EqualTo(expectedRevision));
+                Assert.That(versionInfo.ProductMajorPart, Is.EqualTo(expectedMajor));
+                Assert.That(versionInfo.ProductMinorPart, Is.EqualTo(expectedMinor));
+                Assert.That(versionInfo.ProductBuildPart, Is.EqualTo(expectedBuild));
+                Assert.That(versionInfo.ProductPrivatePart, Is.EqualTo(expectedRevision));
             });
         }
     }
diff --git a/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/StringExtensionsTests.cs b/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/StringExtensionsTests.cs
diff --git a/tests/TestableIO.System.IO.Abstractions.Wrappers.Tests/ConvertersTests.cs b/tests/TestableIO.System.IO.Abstractions.Wrappers.Tests/ConvertersTests.cs

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`[assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]`
`2`		`-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.TestingHelpers.Tests")]`
`3`	`2`	`[assembly: System.Runtime.Versioning.TargetFramework(".NETFramework,Version=v4.7.2", FrameworkDisplayName=".NET Framework 4.7.2")]`
`4`	`3`	`namespace System.IO.Abstractions.TestingHelpers`
`5`	`4`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,5 @@`
`1`	`1`	`[assembly: System.CLSCompliant(true)]`
`2`	`2`	`[assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/TestableIO/System.IO.Abstractions.git")]`
`3`		`-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("DynamicProxyGenAssembly2")]`
`4`		`-[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("System.IO.Abstractions.Tests")]`
`5`	`3`	`[assembly: System.Runtime.Versioning.TargetFramework(".NETFramework,Version=v4.7.2", FrameworkDisplayName=".NET Framework 4.7.2")]`
`6`	`4`	`namespace System.IO.Abstractions`
`7`	`5`	`{`