You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sa...@apache.org on 2012/02/21 20:05:31 UTC

svn commit: r1291958 - /lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestScriptTransformer.java

Author: sarowe
Date: Tue Feb 21 19:05:31 2012
New Revision: 1291958

URL: http://svn.apache.org/viewvc?rev=1291958&view=rev
Log:
Test fix for DIH TestScriptTransformer: Lucene-Solr-Maven-trunk build#397 produced an exception with the message "<script> can be used only in java 6 or above Processing Document # 1", while the test was only ignoring those exceptions with the exact message "<script> can be used only in java 6 or above" - switched the test for this to require the expected message as a prefix rather than exact equality.

Modified:
    lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestScriptTransformer.java

Modified: lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestScriptTransformer.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestScriptTransformer.java?rev=1291958&r1=1291957&r2=1291958&view=diff
==============================================================================
--- lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestScriptTransformer.java (original)
+++ lucene/dev/trunk/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestScriptTransformer.java Tue Feb 21 19:05:31 2012
@@ -53,7 +53,8 @@ public class TestScriptTransformer exten
       sep.applyTransformer(map);
       assertEquals(map.get("name"), "Hello Scott");
     } catch (DataImportHandlerException e) {
-      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", "<script> can be used only in java 6 or above".equals(e.getMessage()));
+      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", 
+                  e.getMessage().startsWith("<script> can be used only in java 6 or above"));      
       throw e;
     }
   }
@@ -85,7 +86,8 @@ public class TestScriptTransformer exten
       sep.applyTransformer(map);
       assertEquals(map.get("name"), "Hello Scott");
     } catch (DataImportHandlerException e) {
-      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", "<script> can be used only in java 6 or above".equals(e.getMessage()));
+      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", 
+                  e.getMessage().startsWith("<script> can be used only in java 6 or above"));      
       throw e;
     }
   }
@@ -101,7 +103,8 @@ public class TestScriptTransformer exten
               .item(0));
       assertTrue(config.script.text.indexOf("checkNextToken") > -1);
     } catch (DataImportHandlerException e) {
-      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", "<script> can be used only in java 6 or above".equals(e.getMessage()));
+      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", 
+                  e.getMessage().startsWith("<script> can be used only in java 6 or above"));      
       throw e;
     }
   }
@@ -129,7 +132,8 @@ public class TestScriptTransformer exten
       sep.applyTransformer(map);
       assertNull(map.get("$hasMore"));
     } catch (DataImportHandlerException e) {
-      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", "<script> can be used only in java 6 or above".equals(e.getMessage()));
+      assumeFalse("JRE does not contain a JavaScript engine (OpenJDK)", 
+                  e.getMessage().startsWith("<script> can be used only in java 6 or above"));      
       throw e;
     }
   }