From ed562bb1ba65a5427decd6712c69b6f4a16849a0 Mon Sep 17 00:00:00 2001
From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com>
Date: Mon, 13 Jan 2025 12:26:28 -0600
Subject: [PATCH] Bump NunitXml.TestLogger from 4.1.0 to 5.0.0 (#2098)

* Bump NunitXml.TestLogger from 4.1.0 to 5.0.0

Bumps [NunitXml.TestLogger](https://github.com/spekt/testlogger) from 4.1.0 to 5.0.0.
- [Release notes](https://github.com/spekt/testlogger/releases)
- [Changelog](https://github.com/spekt/testlogger/blob/master/CHANGELOG.md)
- [Commits](https://github.com/spekt/testlogger/compare/v4.1.0...v5.0.0)

---
updated-dependencies:
- dependency-name: NunitXml.TestLogger
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Remove disused test loggers

* Remove old TestLogger ref from docs

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: James A Sutherland <>
Co-authored-by: James Friel <jfriel001@dundee.ac.uk>
---
 Directory.Packages.props                     |  3 +-
 Documentation/CodeTutorials/PluginWriting.md | 38 ++++++++++----------
 Rdmp.Core.Tests/Rdmp.Core.Tests.csproj       |  1 -
 Rdmp.UI.Tests/Rdmp.UI.Tests.csproj           |  1 -
 4 files changed, 19 insertions(+), 24 deletions(-)

diff --git a/Directory.Packages.props b/Directory.Packages.props
index 9a8ea5c911..13014bbcd4 100644
--- a/Directory.Packages.props
+++ b/Directory.Packages.props
@@ -39,11 +39,10 @@
     <PackageVersion Include="System.Resources.Extensions" Version="9.0.0" />
     <PackageVersion Include="VPKSoft.ScintillaLexers.NET" Version="1.1.16" />
     <PackageVersion Include="HIC.System.Windows.Forms.DataVisualization" Version="1.0.1" />
-    <PackageVersion Include="NunitXml.TestLogger" Version="4.1.0" />
     <PackageVersion Include="NUnit3TestAdapter" Version="4.6.0" />
     <PackageVersion Include="NUnit.Analyzers" Version="4.6.0" />
     <PackageVersion Include="NSubstitute" Version="5.3.0" />
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
     <PackageVersion Include="coverlet.collector" Version="6.0.3" />
   </ItemGroup>
-</Project>
\ No newline at end of file
+</Project>
diff --git a/Documentation/CodeTutorials/PluginWriting.md b/Documentation/CodeTutorials/PluginWriting.md
index 1fe842faf2..9240a2c5db 100644
--- a/Documentation/CodeTutorials/PluginWriting.md
+++ b/Documentation/CodeTutorials/PluginWriting.md
@@ -504,7 +504,6 @@ dotnet add package HIC.RDMP.Plugin.Test
 dotnet add package Microsoft.NET.Test.Sdk
 dotnet add package NUnit
 dotnet add package NUnit3TestAdapter
-dotnet add package NunitXml.TestLogger
 ```
 
 Add the following test:
@@ -516,27 +515,26 @@ using Rdmp.Core.DataFlowPipeline;
 using ReusableLibraryCode.Progress;
 using System.Data;
 
-namespace MyPipelinePluginTests
+namespace MyPipelinePluginTests;
+
+public class TestAnonymisationPlugins
 {
-    public class TestAnonymisationPlugins
+    [Test]
+    public void TestBasicDataTableAnonymiser1()
     {
-        [Test]
-        public void TestBasicDataTableAnonymiser1()
-        {
-            var dt = new DataTable();
-            dt.Columns.Add("Story");
-            dt.Rows.Add(new[] {"Thomas went to school regularly"});
-            dt.Rows.Add(new[] {"It seems like Wallace went less regularly"});
-            dt.Rows.Add(new[] {"Mr Smitty was the teacher"});
-
-            var a = new BasicDataTableAnonymiser1();
-            var resultTable = a.ProcessPipelineData(dt,new ThrowImmediatelyDataLoadEventListener(),new GracefulCancellationToken());
-
-            Assert.AreEqual(resultTable.Rows.Count,3);
-            Assert.AreEqual("REDACTED went to school regularly",resultTable.Rows[0][0]);
-            Assert.AreEqual("It seems like REDACTED went less regularly",resultTable.Rows[1][0]);
-            Assert.AreEqual("Mr Smitty was the teacher",resultTable.Rows[2][0]);
-        }
+        using var dt = new DataTable();
+        dt.Columns.Add("Story");
+        dt.Rows.Add(new[] {"Thomas went to school regularly"});
+        dt.Rows.Add(new[] {"It seems like Wallace went less regularly"});
+        dt.Rows.Add(new[] {"Mr Smitty was the teacher"});
+
+        var a = new BasicDataTableAnonymiser1();
+        var resultTable = a.ProcessPipelineData(dt,new ThrowImmediatelyDataLoadEventListener(),new GracefulCancellationToken());
+
+        Assert.AreEqual(resultTable.Rows.Count,3);
+        Assert.AreEqual("REDACTED went to school regularly",resultTable.Rows[0][0]);
+        Assert.AreEqual("It seems like REDACTED went less regularly",resultTable.Rows[1][0]);
+        Assert.AreEqual("Mr Smitty was the teacher",resultTable.Rows[2][0]);
     }
 }
 
diff --git a/Rdmp.Core.Tests/Rdmp.Core.Tests.csproj b/Rdmp.Core.Tests/Rdmp.Core.Tests.csproj
index cb80e9c6ae..4821571fde 100644
--- a/Rdmp.Core.Tests/Rdmp.Core.Tests.csproj
+++ b/Rdmp.Core.Tests/Rdmp.Core.Tests.csproj
@@ -80,7 +80,6 @@
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="NunitXml.TestLogger" />
     </ItemGroup>
     <ItemGroup>
         <ProjectReference Include="..\Rdmp.Core\Rdmp.Core.csproj" />
diff --git a/Rdmp.UI.Tests/Rdmp.UI.Tests.csproj b/Rdmp.UI.Tests/Rdmp.UI.Tests.csproj
index a99c0723dc..9e4b588f3c 100644
--- a/Rdmp.UI.Tests/Rdmp.UI.Tests.csproj
+++ b/Rdmp.UI.Tests/Rdmp.UI.Tests.csproj
@@ -30,7 +30,6 @@
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="NunitXml.TestLogger" />
     </ItemGroup>
     <ItemGroup>
         <ProjectReference Include="..\Application\ResearchDataManagementPlatform\ResearchDataManagementPlatform.csproj" />