Clean Up PCPATR

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/AndFileLoaderTest.cs

@@ -20,7 +20,7 @@ class AndFileLoaderTest : DisambiguateTests
 	{
 		string AndFile { get; set; }
 
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void LoadAndFileGuidsTest()
 		{
 			AndFile = Path.Combine(TestDataDir, "Text4LoadTest.and");

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/DisambiguateSegmentTest.cs

@@ -51,7 +51,7 @@ public override void FixtureTeardown()
 		/// <summary>
 		/// Test disambiguating segment in a text
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void DisambiguateSegmentTest()
 		{
 			//MyCache = Loader.CreateCache();
@@ -131,7 +131,7 @@ public void DisambiguateSegmentTest()
 		/// <summary>
 		/// Test disambiguating segment in a text
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void EnsureMorphBundleHasSenseTest()
 		{
 			//MyCache = Loader.CreateCache();

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/DisambiguateTest.cs

@@ -41,10 +41,8 @@ public override void FixtureSetup()
 			if (String.IsNullOrEmpty(SavedTestFile))
 				SavedTestFile = kTestFileB4;
 
-			using (var temp = new IO.TempFile(Path.Combine(Path.GetTempPath(), TestFile), false))
-			{
-				TempTestFile = temp.Path;
-			}
+			using var temp = new IO.TempFile(Path.Combine(Path.GetTempPath(), TestFile), false);
+			TempTestFile = temp.Path;
 			File.Copy(Path.Combine(TestDataDir, SavedTestFile), TempTestFile, true);
 			ProjId = new ProjectId(TempTestFile);
 			FwRegistryHelper.Initialize();

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/DisambiguateText2Test.cs

@@ -46,7 +46,7 @@ public override void FixtureTeardown()
 		/// <summary>
 		/// Test disambiguating segment in a text
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void DisambiguateText2Test()
 		{
 			//MyCache = Loader.CreateCache();

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/DisambiguateTextTest.cs

@@ -59,7 +59,7 @@ public override void FixtureTeardown()
 		/// <summary>
 		/// Test disambiguating segment in a text
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void DisambiguateTextTest()
 		{
 			//MyCache = Loader.CreateCache();

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/FLExDBExtractorTests.cs

@@ -52,7 +52,7 @@ public override void FixtureTeardown()
 		/// <summary>
 		/// Test extracting of lexicon.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void ExtractLexiconTest()
 		{
 			//MyCache = Loader.CreateCache();
@@ -66,7 +66,7 @@ public void ExtractLexiconTest()
 			Assert.AreEqual(Lexicon, lexicon);
 		}
 
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void IsAttachedCliticTest()
 		{
 			//MyCache = Loader.CreateCache();
@@ -103,7 +103,7 @@ public void IsAttachedCliticTest()
 			Assert.IsTrue(extractor.IsAttachedClitic(MoMorphTypeTags.kguidMorphProclitic, 2));
 		}
 
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void GetOrComputeWordCategoryTest()
 		{
 			//MyCache = Loader.CreateCache();
@@ -160,7 +160,7 @@ public void GetOrComputeWordCategoryTest()
 		/// <summary>
 		/// Test extracting of text segments in ANA format.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void ExtractTextSegmentAsANATest()
 		{
 			//MyCache = Loader.CreateCache();

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/GuidConverterTest.cs

@@ -15,7 +15,7 @@ namespace SIL.DisambiguateInFLExDBTest
 	[TestFixture]
 	class GuidConverterTest
 	{
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void ConvertGuidsTest()
 		{
 			string result =

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/PcPatrInvokerTests.cs

@@ -27,7 +27,7 @@ class PcPatrInvokerTests : DisambiguateTests
 		/// <summary>
 		/// Test extracting of lexicon.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void PcPatrInvokerTest()
 		{
 			// Check for the existence of the PcPatr executable
@@ -128,7 +128,7 @@ string expectedValue
 		/// <summary>
 		/// Test extracting of lexicon.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void PcPatrInvokerFailureTest()
 		{
 			string grammarFile = "GrammarFail.grm";

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/ToneParsInvokerOptionsTest.cs

@@ -35,7 +35,7 @@ public void FixtureTeardown()
 		/// <summary>
 		/// Test setting of trace and verify options.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void TraceOptionsTest()
 		{
 			Assert.AreEqual("", ToneParsInvokerOptions.Instance.GetOptionsString());

Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/ToneParsInvokerTests.cs

@@ -66,7 +66,7 @@ public override void FixtureTeardown()
 		/// <summary>
 		/// Test conversion of log file using hvos to using glosses.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void ToneParsHvoToGlossInLogTest()
 		{
 			var logFileWithHvos = Path.Combine(TestDataDir, "ToneParsInvokerWithHvos.log");
@@ -83,7 +83,8 @@ public void ToneParsHvoToGlossInLogTest()
 		/// <summary>
 		/// Test invoking of XAmple followed by TonePars.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
+		[Category("DesktopRequired")]
 		public void ToneParsInvokerTest()
 		{
 			// Check for the existence of the TonePars executable

Src/Utilities/pcpatrflex/PrepFLExDBDll/PrepFLExDBTests/PreparerTests.cs

@@ -62,7 +62,7 @@ public override void FixtureTeardown()
 		/// <summary>
 		/// Test we get the expected results for the preparer service.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void PCPATRPreparerTest()
 		{
 			FixtureSetup();
@@ -128,7 +128,7 @@ private void CheckMatch(ICmPossibilityList last, string sToMatch)
 		/// <summary>
 		/// Test we get the expected results for the preparer service.
 		/// </summary>
-		[Test, Ignore("Ignoring this test for timing purposes")]
+		[Test]
 		public void ToneParsPreparerTest()
 		{
 			FixtureSetup();

Test.runsettings

@@ -22,7 +22,7 @@
     <TargetPlatform>x64</TargetPlatform>
     <!-- Framework version - net48 -->
     <TargetFrameworkVersion>net48</TargetFrameworkVersion>
-    <!-- Test timeout in milliseconds (10 minutes default for long-running tests) -->
+	<!-- Test timeout in milliseconds (15 minutes default for long-running tests) -->
 	<TestSessionTimeout>900000</TestSessionTimeout>
     <!--
       InIsolation: Run tests in a separate process.