This is an automated email from the ASF dual-hosted git repository.

rec pushed a commit to branch 
feature/UIMA-5824-Run-JCasUtil-tests-against-equivalent-v3-selects
in repository https://gitbox.apache.org/repos/asf/uima-uimafit.git


The following commit(s) were added to 
refs/heads/feature/UIMA-5824-Run-JCasUtil-tests-against-equivalent-v3-selects 
by this push:
     new d3ca31a  [UIMA-5824] Run JCasUtil tests against equivalent v3 selects
d3ca31a is described below

commit d3ca31aa0e0d5606dbf3864c03e2c3e53036acf5
Author: Richard Eckart de Castilho <[email protected]>
AuthorDate: Wed Jul 25 22:16:52 2018 +0200

    [UIMA-5824] Run JCasUtil tests against equivalent v3 selects
    
    - Adjust to changes from UIMA-5846
---
 .../src/test/java/org/apache/uima/fit/util/JCasUtilv3Test.java     | 7 -------
 1 file changed, 7 deletions(-)

diff --git 
a/uimafit-core/src/test/java/org/apache/uima/fit/util/JCasUtilv3Test.java 
b/uimafit-core/src/test/java/org/apache/uima/fit/util/JCasUtilv3Test.java
index 4ab07f7..a7352b6 100644
--- a/uimafit-core/src/test/java/org/apache/uima/fit/util/JCasUtilv3Test.java
+++ b/uimafit-core/src/test/java/org/apache/uima/fit/util/JCasUtilv3Test.java
@@ -43,7 +43,6 @@ import static org.junit.Assert.fail;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
@@ -239,12 +238,6 @@ public class JCasUtilv3Test extends ComponentTestBase {
 
         ti = System.currentTimeMillis();
         // uimaFIT: selectBetween(Sentence.class, t1, t2);
-        // in uimaFIT, selectBetween always returns annotations in index 
order, so the reverse
-        // if t2 < t1 is not necessary. Adding it here because UIMA v3 between 
works differently.
-        if (t2.getBegin() < t1.getBegin()) {
-          Collections.reverse(reference);
-        }
-        
         List<Sentence> actual = jcas.select(Sentence.class).between(t1, 
t2).asList();
         timeOptimized += System.currentTimeMillis() - ti;
 

Reply via email to