From 85223ce6cb38fca0ff817be84fca3faf679f6373 Mon Sep 17 00:00:00 2001
From: Tom Deseyn <tom.deseyn@gmail.com>
Date: Thu, 30 Jun 2022 18:05:48 +0200
Subject: [PATCH] Reflection.Metadata.Tests: make tests pass when rsa+sha1
 signing is not supported.

---
 .../tests/PortableExecutable/PEBuilderTests.cs                | 4 ++--
 .../tests/System.Reflection.Metadata.Tests.csproj             | 2 ++
 .../tests/TestUtilities/SigningUtilities.cs                   | 3 +++
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/libraries/System.Reflection.Metadata/tests/PortableExecutable/PEBuilderTests.cs b/src/libraries/System.Reflection.Metadata/tests/PortableExecutable/PEBuilderTests.cs
index c3dab37bbd359..33c57919af240 100644
--- a/src/libraries/System.Reflection.Metadata/tests/PortableExecutable/PEBuilderTests.cs
+++ b/src/libraries/System.Reflection.Metadata/tests/PortableExecutable/PEBuilderTests.cs
@@ -135,7 +135,7 @@ public void BasicValidation(Machine machine)
             }
         }
 
-        [Fact]
+        [ConditionalFact(typeof(SigningUtilities), nameof(SigningUtilities.SupportsSigning))]
         [SkipOnPlatform(TestPlatforms.Browser, "System.Security.Cryptography isn't supported on browser")]
         public void BasicValidationSigned()
         {
@@ -748,7 +748,7 @@ private static IEnumerable<string> GetBlobRanges(BlobBuilder builder, IEnumerabl
             }
         }
 
-        [Fact]
+        [ConditionalFact(typeof(SigningUtilities), nameof(SigningUtilities.SupportsSigning))]
         [SkipOnPlatform(TestPlatforms.Browser, "System.Security.Cryptography isn't supported on browser")]
         public void Checksum()
         {
diff --git a/src/libraries/System.Reflection.Metadata/tests/System.Reflection.Metadata.Tests.csproj b/src/libraries/System.Reflection.Metadata/tests/System.Reflection.Metadata.Tests.csproj
index fba5deaa36d76..d611b766fc215 100644
--- a/src/libraries/System.Reflection.Metadata/tests/System.Reflection.Metadata.Tests.csproj
+++ b/src/libraries/System.Reflection.Metadata/tests/System.Reflection.Metadata.Tests.csproj
@@ -9,6 +9,8 @@
     <EnableLibraryImportGenerator>true</EnableLibraryImportGenerator>
   </PropertyGroup>
   <ItemGroup>
+    <Compile Include="$(CommonTestPath)System\Security\Cryptography\SignatureSupport.cs"
+             Link="CommonTest\System\Security\Cryptography\SignatureSupport.cs" />
     <Compile Include="$(CommonPath)Interop\Windows\Interop.Libraries.cs"
              Link="Common\Interop\Windows\Interop.Libraries.cs" />
     <Compile Include="$(CommonPath)Interop\Windows\Kernel32\Interop.FreeLibrary.cs"
diff --git a/src/libraries/System.Reflection.Metadata/tests/TestUtilities/SigningUtilities.cs b/src/libraries/System.Reflection.Metadata/tests/TestUtilities/SigningUtilities.cs
index ac375d87ad78b..2b0d573614c59 100644
--- a/src/libraries/System.Reflection.Metadata/tests/TestUtilities/SigningUtilities.cs
+++ b/src/libraries/System.Reflection.Metadata/tests/TestUtilities/SigningUtilities.cs
@@ -11,6 +11,9 @@ namespace System.Reflection.PortableExecutable.Tests
 {
     internal static class SigningUtilities
     {
+        public static bool SupportsSigning { get; } =
+            System.Security.Cryptography.Tests.SignatureSupport.CanProduceSha1Signature(RSA.Create());
+
         public static byte[] CalculateRsaSignature(IEnumerable<Blob> content, byte[] privateKey)
         {
             var hash = CalculateSha1(content);