You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by sc...@apache.org on 2014/11/15 21:52:56 UTC

svn commit: r1639910 - /uima/uimaj/trunk/uimaj-core/src/test/java/org/apache/uima/cas/test/NewPrimitiveTypesTest.java

Author: schor
Date: Sat Nov 15 20:52:55 2014
New Revision: 1639910

URL: http://svn.apache.org/r1639910
Log:
[UIMA-4059] test did invalid operations on key values after adding FS to indices.  Fix the test. 

Modified:
    uima/uimaj/trunk/uimaj-core/src/test/java/org/apache/uima/cas/test/NewPrimitiveTypesTest.java

Modified: uima/uimaj/trunk/uimaj-core/src/test/java/org/apache/uima/cas/test/NewPrimitiveTypesTest.java
URL: http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-core/src/test/java/org/apache/uima/cas/test/NewPrimitiveTypesTest.java?rev=1639910&r1=1639909&r2=1639910&view=diff
==============================================================================
--- uima/uimaj/trunk/uimaj-core/src/test/java/org/apache/uima/cas/test/NewPrimitiveTypesTest.java (original)
+++ uima/uimaj/trunk/uimaj-core/src/test/java/org/apache/uima/cas/test/NewPrimitiveTypesTest.java Sat Nov 15 20:52:55 2014
@@ -360,6 +360,7 @@ public class NewPrimitiveTypesTest exten
     validateFSData(cas);
 
     // editing the original FS should not change the clone
+    englishView.removeFsFromIndexes(fs);
     fs.setStringValue(stringFeature, "foo");
     fs.setFloatValue(floatFeature, -1f);
     fs.setByteValue(byteFeature, (byte) -1);
@@ -367,7 +368,7 @@ public class NewPrimitiveTypesTest exten
     fs.setShortValue(shortFeature, (short) -1);
     fs.setLongValue(longFeature, -1);
     fs.setDoubleValue(doubleFeature, -1);
-
+    englishView.addFsToIndexes(fs);
     validateFSData(cas);
   }
 
@@ -463,7 +464,6 @@ public class NewPrimitiveTypesTest exten
 
     // create an FS of exampleType and index it
     AnnotationFS fs = englishView.createAnnotation(exampleType, 1, 5);
-    englishView.getIndexRepository().addFS(fs);
 
     // create Array FSs
     StringArrayFS strArrayFS = parmCas.createStringArrayFS(5);
@@ -540,6 +540,8 @@ public class NewPrimitiveTypesTest exten
     fs.setFeatureValue(longArrayFeature, longArrayFS);
     fs.setDoubleValue(doubleFeature, Double.MAX_VALUE);
     fs.setFeatureValue(doubleArrayFeature, doubleArrayFS);
+    
+    englishView.getIndexRepository().addFS(fs);
   }
 
   public void testNewPrimitiveTypeKeys() throws Exception {