NightOwl888 commented on code in PR #1021:
URL: https://github.com/apache/lucenenet/pull/1021#discussion_r1839317334


##########
src/Lucene.Net.TestFramework/Util/LuceneTestCase.cs:
##########
@@ -929,23 +929,35 @@ public virtual void TearDown()
                 */
 
             TestResult result = 
TestExecutionContext.CurrentContext.CurrentResult;
-            string message;
+
             if (result.ResultState == ResultState.Failure || 
result.ResultState == ResultState.Error)
             {
-                message = result.Message + $"\n\nTo reproduce this test 
result:\n\n" +
-                    $"Option 1:\n\n" +
-                    $" Apply the following assembly-level attributes:\n\n" +
-                    $"[assembly: 
Lucene.Net.Util.RandomSeed(\"{RandomizedContext.CurrentContext.RandomSeedAsHex}\")]\n"
 +
-                    $"[assembly: 
NUnit.Framework.SetCulture(\"{Thread.CurrentThread.CurrentCulture.Name}\")]\n\n"
 +
-                    $"Option 2:\n\n" +
-                    $" Use the following .runsettings file:\n\n" +
-                    $"<RunSettings>\n" +
-                    $"  <TestRunParameters>\n" +
-                    $"    <Parameter name=\"tests:seed\" 
value=\"{RandomizedContext.CurrentContext.RandomSeedAsHex}\" />\n" +
-                    $"    <Parameter name=\"tests:culture\" 
value=\"{Thread.CurrentThread.CurrentCulture.Name}\" />\n" +
-                    $"  </TestRunParameters>\n" +
-                    $"</RunSettings>\n\n" +
-                    $"See the .runsettings documentation at: 
https://docs.microsoft.com/en-us/visualstudio/test/configure-unit-tests-by-using-a-dot-runsettings-file.";;
+                string message =
+                    $$"""
+                      {{result.Message}}
+
+                      To reproduce this test result:
+
+                      Option 1:
+
+                       Apply the following assembly-level attributes:
+
+                      [assembly: 
Lucene.Net.Util.RandomSeed("{{RandomizedContext.CurrentContext.RandomSeedAsHex}}")]
+                      [assembly: 
NUnit.Framework.SetCulture("{{Thread.CurrentThread.CurrentCulture.Name}}")]
+
+                      Option 2:

Review Comment:
   Could we just add this as Option 3? The syntax of the `.runsettings` file is 
hard to remember so I would rather not leave it out of the message entirely.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to