Hi Hadrian,

I have found the problem in the first test that failed:
GenericFileDeleteProcessStrategyTest.testCannotDeleteFile

In the method exists file the test checks if the file name equals "target/foo/boom.txt" but on windows the file name is "target\foo\boom.txt". So the method says the file does not exist. The test then thinks it has successfully deleted the file and does not throw the expected exception.

I will report back when I found the problem in the other cases. In any case it would be a good idea to have a windows system that does continouus integration for camel beside the existing linux and mac systems.

Greetings

Christian

Hadrian Zbarcea schrieb:
Hi Christian,

Thanks for doing the tests. No, it is not ok, but all the tests are passing consistently for a few weeks on mac and linux. That code is not quite platform specific (well, except for the Windows fs). Could you *please* take a look a bit more and maybe figure out what the issue is? Is it a fluke, a timing issue, a more serious problem, does it work from an IDE like Eclipse?

Imho this is not a blocker for this release (yet), but it's good to know.

Many thanks again,
Hadrian


On Aug 19, 2009, at 7:26 AM, Christian Schneider wrote:

I just checked out the head (hope this is still the release 2.0.0 state) and did a full build. I got three test failures in camel-core. Is this ok?

I built on Windows Vista, JDK 1.6.0_14 and maven 2.2.0.

Greetings

Christian
Failure Details

<icon_warning_sml.gif>    testCannotDeleteFile

junit.framework.AssertionFailedError: Should have thrown an exception

org.apache.camel.component.file.strategy.GenericFileDeleteProcessStrategyTest:125
<icon_warning_sml.gif>    testCacheStreamToFileAndNotCloseStream

junit.framework.AssertionFailedError: we should have no temp file expected:<1> but was:<0>

org.apache.camel.converter.stream.CachedOutputStreamTest:117
<icon_warning_sml.gif>    testRouteIsCorrectAtRuntime

junit.framework.AssertionFailedError: expected:<4> but was:<1>

org.apache.camel.processor.ChoiceWithEndTest:39


Hadrian Zbarcea schrieb:

The first release candidate of apache-camel-2.0.0 is out with 61 new features, improvements and bug fixes. This concludes more than one year of efforts to bring a new and improved Apache Camel. Many thanks to everybody in the community who contributed to this release.

Please find the staging repo here:
https://repository.apache.org/content/groups/staging/org/apache/camel/

The tarballs are here
https://repository.apache.org/content/groups/staging/org/apache/camel/apache-camel/2.0.0/

Please vote to approve this release binary

[ ] +1 Release the binary as Apache Camel 2.0-M3
[ ] -1 Veto the release (provide specific comments)
Vote is open for 72 hours.


The Camel Riders




Reply via email to