Skip to content

Commit d7fb1bc

Browse files
committed
Revert "LT-21804: try ignoring new utility tests to avoid test failure"
This reverts commit 6d235fa.
1 parent 6d235fa commit d7fb1bc

11 files changed

Lines changed: 19 additions & 21 deletions

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class AndFileLoaderTest : DisambiguateTests
2020
{
2121
string AndFile { get; set; }
2222

23-
[Test, Ignore("Ignoring this test for timing purposes")]
23+
[Test]
2424
public void LoadAndFileGuidsTest()
2525
{
2626
AndFile = Path.Combine(TestDataDir, "Text4LoadTest.and");

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public override void FixtureTeardown()
5151
/// <summary>
5252
/// Test disambiguating segment in a text
5353
/// </summary>
54-
[Test, Ignore("Ignoring this test for timing purposes")]
54+
[Test]
5555
public void DisambiguateSegmentTest()
5656
{
5757
//MyCache = Loader.CreateCache();
@@ -131,7 +131,7 @@ public void DisambiguateSegmentTest()
131131
/// <summary>
132132
/// Test disambiguating segment in a text
133133
/// </summary>
134-
[Test, Ignore("Ignoring this test for timing purposes")]
134+
[Test]
135135
public void EnsureMorphBundleHasSenseTest()
136136
{
137137
//MyCache = Loader.CreateCache();

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

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,8 @@ public override void FixtureSetup()
4141
if (String.IsNullOrEmpty(SavedTestFile))
4242
SavedTestFile = kTestFileB4;
4343

44-
using (var temp = new IO.TempFile(Path.Combine(Path.GetTempPath(), TestFile), false))
45-
{
46-
TempTestFile = temp.Path;
47-
}
44+
var temp = new IO.TempFile(Path.Combine(Path.GetTempPath(), TestFile), false);
45+
TempTestFile = temp.Path;
4846
File.Copy(Path.Combine(TestDataDir, SavedTestFile), TempTestFile, true);
4947
ProjId = new ProjectId(TempTestFile);
5048
FwRegistryHelper.Initialize();

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public override void FixtureTeardown()
4646
/// <summary>
4747
/// Test disambiguating segment in a text
4848
/// </summary>
49-
[Test, Ignore("Ignoring this test for timing purposes")]
49+
[Test]
5050
public void DisambiguateText2Test()
5151
{
5252
//MyCache = Loader.CreateCache();

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public override void FixtureTeardown()
5959
/// <summary>
6060
/// Test disambiguating segment in a text
6161
/// </summary>
62-
[Test, Ignore("Ignoring this test for timing purposes")]
62+
[Test]
6363
public void DisambiguateTextTest()
6464
{
6565
//MyCache = Loader.CreateCache();

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public override void FixtureTeardown()
5252
/// <summary>
5353
/// Test extracting of lexicon.
5454
/// </summary>
55-
[Test, Ignore("Ignoring this test for timing purposes")]
55+
[Test]
5656
public void ExtractLexiconTest()
5757
{
5858
//MyCache = Loader.CreateCache();
@@ -66,7 +66,7 @@ public void ExtractLexiconTest()
6666
Assert.AreEqual(Lexicon, lexicon);
6767
}
6868

69-
[Test, Ignore("Ignoring this test for timing purposes")]
69+
[Test]
7070
public void IsAttachedCliticTest()
7171
{
7272
//MyCache = Loader.CreateCache();
@@ -103,7 +103,7 @@ public void IsAttachedCliticTest()
103103
Assert.IsTrue(extractor.IsAttachedClitic(MoMorphTypeTags.kguidMorphProclitic, 2));
104104
}
105105

106-
[Test, Ignore("Ignoring this test for timing purposes")]
106+
[Test]
107107
public void GetOrComputeWordCategoryTest()
108108
{
109109
//MyCache = Loader.CreateCache();
@@ -160,7 +160,7 @@ public void GetOrComputeWordCategoryTest()
160160
/// <summary>
161161
/// Test extracting of text segments in ANA format.
162162
/// </summary>
163-
[Test, Ignore("Ignoring this test for timing purposes")]
163+
[Test]
164164
public void ExtractTextSegmentAsANATest()
165165
{
166166
//MyCache = Loader.CreateCache();

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ namespace SIL.DisambiguateInFLExDBTest
1515
[TestFixture]
1616
class GuidConverterTest
1717
{
18-
[Test, Ignore("Ignoring this test for timing purposes")]
18+
[Test]
1919
public void ConvertGuidsTest()
2020
{
2121
string result =

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class PcPatrInvokerTests : DisambiguateTests
2727
/// <summary>
2828
/// Test extracting of lexicon.
2929
/// </summary>
30-
[Test, Ignore("Ignoring this test for timing purposes")]
30+
[Test]
3131
public void PcPatrInvokerTest()
3232
{
3333
// Check for the existence of the PcPatr executable
@@ -128,7 +128,7 @@ string expectedValue
128128
/// <summary>
129129
/// Test extracting of lexicon.
130130
/// </summary>
131-
[Test, Ignore("Ignoring this test for timing purposes")]
131+
[Test]
132132
public void PcPatrInvokerFailureTest()
133133
{
134134
string grammarFile = "GrammarFail.grm";

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public void FixtureTeardown()
3535
/// <summary>
3636
/// Test setting of trace and verify options.
3737
/// </summary>
38-
[Test, Ignore("Ignoring this test for timing purposes")]
38+
[Test]
3939
public void TraceOptionsTest()
4040
{
4141
Assert.AreEqual("", ToneParsInvokerOptions.Instance.GetOptionsString());

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ public override void FixtureTeardown()
6666
/// <summary>
6767
/// Test conversion of log file using hvos to using glosses.
6868
/// </summary>
69-
[Test, Ignore("Ignoring this test for timing purposes")]
69+
[Test]
7070
public void ToneParsHvoToGlossInLogTest()
7171
{
7272
var logFileWithHvos = Path.Combine(TestDataDir, "ToneParsInvokerWithHvos.log");
@@ -83,7 +83,7 @@ public void ToneParsHvoToGlossInLogTest()
8383
/// <summary>
8484
/// Test invoking of XAmple followed by TonePars.
8585
/// </summary>
86-
[Test, Ignore("Ignoring this test for timing purposes")]
86+
[Test]
8787
public void ToneParsInvokerTest()
8888
{
8989
// Check for the existence of the TonePars executable

0 commit comments

Comments
 (0)