Skip to content

Commit

Permalink
remove FIXME's that are no longer needed.
Browse files Browse the repository at this point in the history
  • Loading branch information
johnml1135 committed Oct 31, 2024
1 parent fa6a04d commit ea59cd7
Showing 1 changed file with 0 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ public async Task RunAsync_PretranslateAll()

await env.RunBuildJobAsync(corpus1);

// FIXME This should be 4, but the "don't pretranslate things trained on" logic is not implemented yet.
Assert.That(await env.GetPretranslateCountAsync(), Is.EqualTo(2));
}

Expand Down Expand Up @@ -190,7 +189,6 @@ public async Task RunAsync_MixedSource_Paratext()
Assert.That(trgCount, Is.EqualTo(1));
Assert.That(termCount, Is.EqualTo(0));
});
// FIXME - this should be 56 (or double check)
Assert.That(
await env.GetPretranslateCountAsync(),
Is.EqualTo(13),
Expand All @@ -214,7 +212,6 @@ public async Task RunAsync_MixedSource_Text()
Assert.That(trgCount, Is.EqualTo(1));
Assert.That(termCount, Is.EqualTo(0));
});
// FIXME this should be 9.
Assert.That(
await env.GetPretranslateCountAsync(),
Is.EqualTo(2),
Expand Down Expand Up @@ -482,7 +479,6 @@ await env.GetTargetExtractAsync(),
});
JsonArray? pretranslations = await env.GetPretranslationsAsync();
Assert.That(pretranslations, Is.Not.Null);
// FIXME this should be 37.
Assert.That(pretranslations!.Count, Is.EqualTo(7), pretranslations.ToJsonString());
Assert.That(
pretranslations[2]!["translation"]!.ToString(),
Expand Down

0 comments on commit ea59cd7

Please sign in to comment.