Mailing-List: contact derby-dev-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: <derby-dev@db.apache.org>
Message-ID: <1010660611.1142849583453.JavaMail.jira@ajax>
Date: Mon, 20 Mar 2006 10:13:03 +0000 (GMT)
From: "Kristian Waagan (JIRA)" <derby-dev@db.apache.org>
To: derby-dev@db.apache.org
Subject: [jira] Updated: (DERBY-1097) Add tests for Statement.isClosed()
In-Reply-To: <2085899103.1142009858952.JavaMail.jira@ajax>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

     [ http://issues.apache.org/jira/browse/DERBY-1097?page=all ]

Kristian Waagan updated DERBY-1097:
-----------------------------------

    Attachment: DERBY-1097-2a.stat
                DERBY-1097-2a.diff

Answering David's comments of 18th of March, and uploading 'DERBY-1097-2a.diff' (patch deprecates previous patches).

Committing sound test code that is known to fail beacause of another bug in Derby must be allowed in my opinion, as long as they are not included in any suites and will be fixed shortly. Also, in this case, the fact that the test code works for embedded is an indication (but not a guarantee) that the test code is sound - the same code is run for DerbyNetClient. The error happens in the one-time setup code in StatementTestSetup.setUp(), but it is now gone because the bug is already fixed.
Finally, I have not submitted a patch for enabling the test, thus it would not cause problems for anyone.

I have removed the exclamation marks in the assertion messages.

I don't understand this comment: "It's a little odd to me that you run an embedded test under the DerbyNetClient framework."
Did you look at the old/initial submission, or have I forgotten to remove some comments?
I agree that what the jdbc4 suite does is odd, but that is not my code. I initially wrote StatementTest so that it could be added to the jdbc4 suite, but was convinced this was not the best idea because of how it is run.
StatementTest itself can be run under embedded or DerbyNetClient. It doesn't run under DerbyNet (JCC), because the driver does not support JDBC4.

A new suite will be created, but I'm not sure what is the best solution yet, so I would like some comments on it.
Should I create a new JDBC4 suite, say 'jdbc40', then add this to derbyall and derbynetclientmats?
All JVMs except JDK 1.6 will be excluded and so will running under DerbyNet be.


> Add tests for Statement.isClosed()
> ----------------------------------
>
>          Key: DERBY-1097
>          URL: http://issues.apache.org/jira/browse/DERBY-1097
>      Project: Derby
>         Type: Sub-task
>   Components: Test
>     Versions: 10.2.0.0
>  Environment: JDBC4 / JDK 1.6
>     Reporter: Kristian Waagan
>     Assignee: Kristian Waagan
>  Attachments: DERBY-1097-2a.diff, DERBY-1097-2a.stat, StatementTest.java, StatementTest.java-v1, StatementTestSetup.java, StatementTestSetup.java-v1
>
> Add tests for Statement.isClosed() (implemented as part of DERBY-953).
> The tests are already written, but because of some "confusion" and diverging paths regarding how to get connections when running JUnit tests, they are held back.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira