From 99f5495b1a4a537e2188b5dae8e366129da88820 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Michal=20Strehovsk=C3=BD?=
 <MichalStrehovsky@users.noreply.github.com>
Date: Thu, 2 Feb 2023 06:28:09 +0900
Subject: [PATCH] Do not remove resource messages in Xml tests (#81463)

Addresses first bullet of #81455.

Addresses 211 test failures to the tune of:

```
[FAIL] System.Xml.XmlSchemaValidatorApiTests.TCValidateWhitespace_String.WhitespaceInsideElement
System.Xml.Tests.VerifyException : GetManifestResourceStream() failed
   at System.Xml.Tests.ExceptionVerifier..ctor(String assemblyName, ExceptionVerificationFlags flags, ITestOutputHelper output) + 0x4c9
   at System.Xml.XmlSchemaValidatorApiTests.TCValidateWhitespace_String..ctor(ITestOutputHelper output) + 0x48
```
---
 .../tests/System.Private.Xml.Tests.csproj                |  4 ++++
 src/libraries/System.Private.Xml/tests/default.rd.xml    | 16 ++++++++++++++++
 2 files changed, 20 insertions(+)
 create mode 100644 src/libraries/System.Private.Xml/tests/default.rd.xml

diff --git a/src/libraries/System.Private.Xml/tests/System.Private.Xml.Tests.csproj b/src/libraries/System.Private.Xml/tests/System.Private.Xml.Tests.csproj
index 2b7df48..f076563 100644
--- a/src/libraries/System.Private.Xml/tests/System.Private.Xml.Tests.csproj
+++ b/src/libraries/System.Private.Xml/tests/System.Private.Xml.Tests.csproj
@@ -7,6 +7,10 @@
     <IlcTrimMetadata>false</IlcTrimMetadata>
   </PropertyGroup>
 
+  <ItemGroup>
+    <RdXmlFile Include="default.rd.xml" />
+  </ItemGroup>
+
   <PropertyGroup Condition="'$(TargetOS)' == 'browser'">
     <!-- This doesn't run on V8 shell because https://bugs.chromium.org/p/v8/issues/detail?id=12541 -->
     <Scenario>WasmTestOnBrowser</Scenario>
diff --git a/src/libraries/System.Private.Xml/tests/default.rd.xml b/src/libraries/System.Private.Xml/tests/default.rd.xml
new file mode 100644
index 0000000..8177c40
--- /dev/null
+++ b/src/libraries/System.Private.Xml/tests/default.rd.xml
@@ -0,0 +1,16 @@
+<Directives>
+  <Application>
+    <Assembly Name="System.Private.Xml">
+      <Type Name="System.SR">
+
+        <!--
+          This method is not actually reflected on by the tests, but its presence
+          tells the compiler not to optimize away the manifest resource that contains
+          resource strings. The tests do reflect on that one in the ExceptionVerifier class.   
+        -->
+        <Method Name="GetResourceString" Dynamic="Required All" />
+
+      </Type>
+    </Assembly>
+  </Application>
+</Directives>
-- 
2.7.4