Mailing-List: contact derby-commits-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Derby Development" <derby-dev@db.apache.org>
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
From: Apache Wiki <wikidiffs@apache.org>
To: derby-commits@db.apache.org
Date: Thu, 05 Jul 2007 17:48:52 -0000
Message-ID: <20070705174852.23525.43772@eos.apache.org>
Subject: [Db-derby Wiki] Update of "ConvertOldTestToJunitTips" by
 KatheyMarsden

Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Db-derby Wiki" for change notification.

The following page has been changed by KatheyMarsden:
http://wiki.apache.org/db-derby/ConvertOldTestToJunitTips

------------------------------------------------------------------------------
       Normally to get a connection to the default database, you should use [http://db.apache.org/derby/javadoc/testing/org/apache/derbyTesting/junit/BaseJDBCTestCase.html#getConnection() getConnection()] This is a single connection that is reused so if you call getConnection twice, you will get the same connection.  If you want a secondary connection, use [http://db.apache.org/derby/javadoc/testing/org/apache/derbyTesting/junit/BaseJDBCTestCase.html#openDefaultConnection() openDefaultConnection]
     * Not removing the old test files. 
       You should remove the old test files, canons and remove the old test from its suite. This includes references in any junit *!HarnessJavaTest classes. ant clobbber before you build, to make sure their are no residual dependencies on the old test files.
-  
+    * Omitting the fail after statement execution if a statement is expected to fail.  The following code would not fail if the statement succeeds.
+ {{{
+ try {
+   s.execute(command);
+ }
+ catch (SQLException e)
+ {
+    assertSQLState("42502", e);
+ }
+  }}}
+ 
+ It should be:
+ {{{
+ try {
+   s.execute(command);
+   fail("Command expected to fail" + command);
+ }
+ catch (SQLException e)
+ {
+    assertSQLState("42502", e);
+ }
+  }}}
+ 
+ or better
+ {{{
+    assertStatementError("42502",s,command);
+ }}}