[ 
https://issues.apache.org/jira/browse/DERBY-3156?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vemund Østgaard updated DERBY-3156:
-----------------------------------

    Attachment: 3156-diffv4.stat
                3156-diffv4

Thanks again Knut, made a new patch and hope it works better now. It had worked 
standalone until I made the last change in the previous patch, of course... :)

Also removed the deleteFileOnExit() method.

About the IOExceptions I'm not really sure. This:

public Object run() throws IOException { 

will not compile if the code in run() suddenly starts throwing other 
exceptions, so we shouldn't be getting any ClassCastException from the catch() 
all of a sudden. If it did change to throw other exceptions you might want 
these assertmethods to throw both exceptions without any wrapping to make it 
possible to make different choices easily. I also think the methods are more 
useful when they wrap away the uninteresting PrivilegedActionException for you.

> Convert testing of derby error stream to JUnit
> ----------------------------------------------
>
>                 Key: DERBY-3156
>                 URL: https://issues.apache.org/jira/browse/DERBY-3156
>             Project: Derby
>          Issue Type: Test
>          Components: Test
>            Reporter: Vemund Østgaard
>            Assignee: Vemund Østgaard
>            Priority: Minor
>         Attachments: 3156-diff, 3156-diff.stat, 3156-diffv2, 
> 3156-diffv2.stat, 3156-diffv3, 3156-diffv3.stat, 3156-diffv4, 3156-diffv4.stat
>
>
> Convert the tests that are related to derby error stream properties and 
> functionality.
> Seems to be these two tests:
> lang/logStream.java 
> lang/errorStream.java

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to