You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Kristian Waagan (JIRA)" <ji...@apache.org> on 2008/07/23 08:48:31 UTC
[jira] Commented: (DERBY-3784) ImportExportTest doesn't detect lack
of expected errors
[ https://issues.apache.org/jira/browse/DERBY-3784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12615896#action_12615896 ]
Kristian Waagan commented on DERBY-3784:
----------------------------------------
I applied the patch and ran the three tests. No new failures.
+1 to commit.
> ImportExportTest doesn't detect lack of expected errors
> -------------------------------------------------------
>
> Key: DERBY-3784
> URL: https://issues.apache.org/jira/browse/DERBY-3784
> Project: Derby
> Issue Type: Bug
> Components: Test
> Affects Versions: 10.5.0.0
> Reporter: Knut Anders Hatlen
> Assignee: Knut Anders Hatlen
> Priority: Minor
> Attachments: d3784.diff
>
>
> I observed that ImportExportTest contains many test cases which do try/catch/assertSQLException, but none of them call fail() at the end of the try block. Example:
> public void testEarlyEndOfFile() throws Exception {
> Connection c = getConnection();
> try {
> doImportFromFile(c, "extin/EndOfFile.txt" , "T4" , null , null , null, 0);
> } catch (SQLException e) {
> assertSQLState("XIE0E", e);
> }
> }
> I think this means that if the expected exception isn't thrown the test still passes.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.