Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Received-SPF: pass (nike.apache.org: domain of hekonsek@gmail.com designates
 209.85.161.173 as permitted sender)
MIME-Version: 1.0
Date: Wed, 6 Jun 2012 08:34:38 +0200
Message-ID: 
 <CALhQg-ZgYknZ0VE6EsWz30PhzfwLT2ZEc+9j3C6bOO+d_jtqOw@mail.gmail.com>
Subject: Improved the test coverage of Camel Stream component
From: Henryk Konsek <hekonsek@gmail.com>
To: dev@camel.apache.org
Content-Type: text/plain; charset=ISO-8859-1

Hi,

I've improved the test coverage of Camel Stream component.

In particular I've added:

a) tests for Stream producer writing to the URL output stream (StreamToUrlTest).

b) assertions to StreamSystemOutTest. Previously we've been sending a
messages to the System.out but made no assertions about it. This
looked like the AssertionFreeTesting [1]. :) I changed the test to
temporarily switch System.out with another PrintStream instance so we
could make some assertions on it.

Could somebody take a look at patch and possibly apply it to the
trunk? Here is Jira ticket [2] for this issue.

Best regards.

[1] http://martinfowler.com/bliki/AssertionFreeTesting.html
[2] https://issues.apache.org/jira/browse/CAMEL-5336

-- 
Henryk Konsek
http://henryk-konsek.blogspot.com