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

Babak Vahdat reopened CAMEL-4255:
---------------------------------


As part of the changes provided in the patch regarding 
TestSupport.deleteDirectory() method was not taken over I reopen the ticket to 
make it sure it was really on purpose. If so feel free to close the ticket 
again. However to my understanding we should not simply ignore/neglect the 
returned boolean value of java.io.File.delete(). In my case it was really 
life-saver to check that, see my comments here working with Hsql while 
unit-testing my routes:
http://camel.465427.n5.nabble.com/Favor-static-member-classes-over-nonstatic-tp4634964p4635206.html

> avoid using "\n" as line seperator while doing I/O and preferably stick to 
> System.getProperty("line.separator")
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-4255
>                 URL: https://issues.apache.org/jira/browse/CAMEL-4255
>             Project: Camel
>          Issue Type: Improvement
>            Reporter: Babak Vahdat
>            Assignee: Claus Ibsen
>            Priority: Minor
>             Fix For: 2.9.0
>
>         Attachments: cleanup.txt
>
>
> See 
> http://camel.465427.n5.nabble.com/VOTE-Release-Apache-Camel-2-8-0-tp4610671p4616161.html
> Other than that consider the evaluation of java.io.File.delete() returned 
> boolean value while unit-testing, to make sure a file/directory get's deleted 
> when we ask for it.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to