From 57b759da66a2673e02b6663830fa86bb4ed5d1f1 Mon Sep 17 00:00:00 2001
From: Enkidu93 <lowryec17@gcc.edu>
Date: Fri, 8 Nov 2024 16:48:01 -0500
Subject: [PATCH] Change naming to avoid confusion with 'Select'

---
 src/SIL.Machine/Corpora/CorporaExtensions.cs           |  4 ++--
 .../Corpora/CorporaExtensionsTests.cs                  | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/SIL.Machine/Corpora/CorporaExtensions.cs b/src/SIL.Machine/Corpora/CorporaExtensions.cs
index a737d8c1..a9e2b4f9 100644
--- a/src/SIL.Machine/Corpora/CorporaExtensions.cs
+++ b/src/SIL.Machine/Corpora/CorporaExtensions.cs
@@ -377,12 +377,12 @@ public static ITextCorpus FilterTexts(this ITextCorpus corpus, IEnumerable<strin
             return new FilterTextsTextCorpus(corpus, textIds);
         }
 
-        public static ITextCorpus SelectRandom(this NParallelTextCorpus corpus, int seed)
+        public static ITextCorpus ChooseRandom(this NParallelTextCorpus corpus, int seed)
         {
             return new MergedCorpus(corpus, MergeRule.Random, seed);
         }
 
-        public static ITextCorpus SelectFirst(this NParallelTextCorpus corpus)
+        public static ITextCorpus ChooseFirst(this NParallelTextCorpus corpus)
         {
             return new MergedCorpus(corpus, MergeRule.First, 0);
         }
diff --git a/tests/SIL.Machine.Tests/Corpora/CorporaExtensionsTests.cs b/tests/SIL.Machine.Tests/Corpora/CorporaExtensionsTests.cs
index 5f397bc3..708b4ffa 100644
--- a/tests/SIL.Machine.Tests/Corpora/CorporaExtensionsTests.cs
+++ b/tests/SIL.Machine.Tests/Corpora/CorporaExtensionsTests.cs
@@ -95,7 +95,7 @@ public void MergedCorpus_SelectFirst()
             )
         );
         var nParallelCorpus = new NParallelTextCorpus([corpus1, corpus2, corpus3]) { AllRowsList = [true, true, true] };
-        var mergedCorpus = nParallelCorpus.SelectFirst();
+        var mergedCorpus = nParallelCorpus.ChooseFirst();
         var rows = mergedCorpus.ToArray();
         Assert.That(rows, Has.Length.EqualTo(3), JsonSerializer.Serialize(rows));
         Assert.That(rows[0].Text, Is.EqualTo("source 1 segment 1 ."));
@@ -140,7 +140,7 @@ public void MergedCorpus_SelectRandom_Seed123456()
             )
         );
         var nParallelCorpus = new NParallelTextCorpus([corpus1, corpus2, corpus3]) { AllRowsList = [true, true, true] };
-        var mergedCorpus = nParallelCorpus.SelectRandom(123456);
+        var mergedCorpus = nParallelCorpus.ChooseRandom(123456);
         var rows = mergedCorpus.ToArray();
         Assert.That(rows, Has.Length.EqualTo(3), JsonSerializer.Serialize(rows));
         Assert.Multiple(() =>
@@ -188,7 +188,7 @@ public void MergedCorpus_SelectRandom_Seed4501()
             )
         );
         var nParallelCorpus = new NParallelTextCorpus([corpus1, corpus2, corpus3]) { AllRowsList = [true, true, true] };
-        var mergedCorpus = nParallelCorpus.SelectRandom(4501);
+        var mergedCorpus = nParallelCorpus.ChooseRandom(4501);
         var rows = mergedCorpus.ToArray();
         Assert.That(rows, Has.Length.EqualTo(3), JsonSerializer.Serialize(rows));
         Assert.Multiple(() =>
@@ -238,7 +238,7 @@ public void AlignMergedCorpora()
 
         ITextCorpus sourceCorpus = (new ITextCorpus[] { sourceCorpus1, sourceCorpus1, sourceCorpus3 })
             .AlignMany([true, true, true])
-            .SelectFirst();
+            .ChooseFirst();
 
         var targetCorpus1 = new DictionaryTextCorpus(
             new MemoryText(
@@ -276,7 +276,7 @@ public void AlignMergedCorpora()
 
         ITextCorpus targetCorpus = (new ITextCorpus[] { targetCorpus1, targetCorpus2, targetCorpus3 })
             .AlignMany([true, true, true])
-            .SelectFirst();
+            .ChooseFirst();
 
         IParallelTextCorpus alignedCorpus = sourceCorpus.AlignRows(targetCorpus);
         ParallelTextRow[] rows = alignedCorpus.GetRows().ToArray();