You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by us...@apache.org on 2017/02/19 11:21:11 UTC

lucene-solr:master: SOLR-9966: Do test-ignore properly

Repository: lucene-solr
Updated Branches:
  refs/heads/master a88e6e563 -> 7e30fe147


SOLR-9966: Do test-ignore properly


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/7e30fe14
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/7e30fe14
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/7e30fe14

Branch: refs/heads/master
Commit: 7e30fe147193105ec36265b03527142e48623a18
Parents: a88e6e5
Author: Uwe Schindler <us...@apache.org>
Authored: Sun Feb 19 12:20:56 2017 +0100
Committer: Uwe Schindler <us...@apache.org>
Committed: Sun Feb 19 12:20:56 2017 +0100

----------------------------------------------------------------------
 .../org/apache/solr/handler/dataimport/TestJdbcDataSource.java    | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/7e30fe14/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestJdbcDataSource.java
----------------------------------------------------------------------
diff --git a/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestJdbcDataSource.java b/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestJdbcDataSource.java
index aed2ed4..2f6b24c 100644
--- a/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestJdbcDataSource.java
+++ b/solr/contrib/dataimporthandler/src/test/org/apache/solr/handler/dataimport/TestJdbcDataSource.java
@@ -485,8 +485,7 @@ public class TestJdbcDataSource extends AbstractDataImportHandlerTestCase {
 
   @Test
   public void testRetrieveFromDriverManager() throws Exception {
-    // in JDK9, Class.forName will throw exception for mock classes
-    if (Constants.JRE_IS_MINIMUM_JAVA9) return;
+    assumeFalse("In Java 9, Class.forName() does not work for mock classes", Constants.JRE_IS_MINIMUM_JAVA9);
     DriverManager.registerDriver(driver);
     try {
       when(driver.connect(notNull(),notNull())).thenReturn(connection);