db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel John Debrunner <...@apache.org>
Subject Re: [jira] Commented: (DERBY-2458) Convert lang/unaryArithmeticDynamicParamter.java to junit
Date Mon, 19 Mar 2007 20:30:44 GMT
Manjula Kutty (JIRA) wrote:
>     [ https://issues.apache.org/jira/browse/DERBY-2458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12482190

>>> The comments don't indicate why this test requires JDBC 3? I couldn't see anything
obvious that would stop this running on JSR169. 
> Added comments to to reason why we need JDBC3. Also if I do specify only about JDBC3,
I thought the test will run in JSR 169 (I may be wrong here...Please clear me if I'm wrong).

I added comments a couple of days ago to the JDBC class to try and 
clarify this. But no, vmSupportsJDBC3() says that this virtual machine 
supports JDBC 3 in full. Since JSR 169 is a subset of JDBC 3 it is not 
true that a virtual machine supporting JSR 169 supports the full JDBC 3 

>>> It looks like the suite method runs the fixtures twice in embedded, once adding
them directly and once through defaultSuite 
> I thought I give 
> return   TestConfiguration.defaultSuite(UnaryArithmeticParameterTest.class);
> The test will run both embedded and n/w server mode. (Please correct me if I'm wrong)

OK, I got confused by your suite method. In the path that returns using 
defaultSuite() you also do create a new suite called 'suite'. This suite 
is never used though, so that code can be removed. Since the class 
didn't compile I couldn't check to see which tests were created.

+	public static Test suite() {
+		if ( !JDBC.vmSupportsJDBC3())
+            return new TestSuite("empty UnaryArithmeticParameterTest - 
supported only from JDBC3.0 and above");
+        else{
+            	TestSuite suite = new 
+            	suite.addTestSuite(UnaryArithmeticParameterTest.class);
+        		//To run the test in both embedded and client/server mode
+		return 
+        }


View raw message