[Jiemamy-notify:2098] commit [3194] refactor

Back to archive index

svnno****@sourc***** svnno****@sourc*****
2009年 4月 8日 (水) 21:51:34 JST


Revision: 3194
          http://svn.sourceforge.jp/view?root=jiemamy&view=rev&rev=3194
Author:   daisuke_m
Date:     2009-04-08 21:51:34 +0900 (Wed, 08 Apr 2009)

Log Message:
-----------
refactor

Modified Paths:
--------------
    artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/DatabaseTest.java
    artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/TestDatabaseInstanceTest.java


-------------- next part --------------
Modified: artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/DatabaseTest.java
===================================================================
--- artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/DatabaseTest.java	2009-04-08 12:29:04 UTC (rev 3193)
+++ artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/DatabaseTest.java	2009-04-08 12:51:34 UTC (rev 3194)
@@ -19,9 +19,12 @@
 package org.jiemamy;
 
 import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.not;
 import static org.hamcrest.Matchers.notNullValue;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
+import static org.junit.Assert.fail;
+import static org.junit.Assume.assumeThat;
 
 import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
@@ -118,15 +121,17 @@
 		logger.info("== Database: " + testDatabase.name());
 		
 		// FIXME Oracleのdialectが整備されるまでskip
-		if (testDatabase == TestDatabaseInstance.Oracle) {
-			logger.warn(testDatabase.name() + " skip");
-			return;
-		}
-		// skip処理ここまで
+		assumeThat(testDatabase, is(not(TestDatabaseInstance.Oracle)));
 		
 		Jiemamy jiemamy = builder.getBuiltModel();
 		
-		doDatabaseTest(testDatabase, jiemamy);
+		try {
+			doDatabaseTest(testDatabase, jiemamy);
+		} catch (Exception e) {
+			String message = "failed on " + testDatabase.name() + " with " + builder.name();
+			logger.error(message, e);
+			fail(message);
+		}
 	}
 	
 	/**
@@ -141,11 +146,7 @@
 		logger.info("== Database: " + testDatabase.name());
 		
 		// FIXME Oracleのdialectが整備されるまでskip
-		if (testDatabase == TestDatabaseInstance.Oracle) {
-			logger.warn(testDatabase.name() + " skip");
-			return;
-		}
-		// skip処理ここまで
+		assumeThat(testDatabase, is(not(TestDatabaseInstance.Oracle)));
 		
 		Jiemamy jiemamy = Jiemamy.newInstance();
 		InputStream in = null;
@@ -156,7 +157,13 @@
 			IOUtils.closeQuietly(in);
 		}
 		
-		doDatabaseTest(testDatabase, jiemamy);
+		try {
+			doDatabaseTest(testDatabase, jiemamy);
+		} catch (Exception e) {
+			String message = "failed on " + testDatabase.name() + " with " + file.getAbsolutePath();
+			logger.error(message, e);
+			fail(message);
+		}
 	}
 	
 	/**

Modified: artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/TestDatabaseInstanceTest.java
===================================================================
--- artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/TestDatabaseInstanceTest.java	2009-04-08 12:29:04 UTC (rev 3193)
+++ artemis/trunk/jiemamy-artemis-test/src/test/java/org/jiemamy/TestDatabaseInstanceTest.java	2009-04-08 12:51:34 UTC (rev 3194)
@@ -21,6 +21,7 @@
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.CoreMatchers.notNullValue;
 import static org.junit.Assert.assertThat;
+import static org.junit.Assert.fail;
 import static org.junit.Assume.assumeThat;
 
 import java.io.File;
@@ -59,13 +60,13 @@
 	/**
 	 * 全てのテストDBに接続できること。
 	 * 
-	 * @param db テストDB
+	 * @param testDatabase テストDB
 	 * @throws Exception 例外が発生した場合
 	 */
 	@Theory
-	public void test01_全てのテストDBに接続できること(TestDatabaseInstance db) throws Exception {
-		logger.info(db.name());
-		DatabaseConnectionConfig config = db.getConfig();
+	public void test01_全てのテストDBに接続できること(TestDatabaseInstance testDatabase) throws Exception {
+		logger.info(testDatabase.name());
+		DatabaseConnectionConfig config = testDatabase.getConfig();
 		Properties props = new Properties();
 		props.setProperty("user", config.getUsername());
 		props.setProperty("password", config.getPassword());
@@ -82,8 +83,9 @@
 			assertThat(connection, is(notNullValue()));
 			logger.info("success");
 		} catch (Exception e) {
-			logger.error("failed", e);
-			throw e;
+			String message = "failed on " + testDatabase.name();
+			logger.error(message, e);
+			fail(message);
 		} finally {
 			JmIOUtil.closeQuietly(connection);
 		}



Jiemamy-notify メーリングリストの案内
Back to archive index