You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ni...@apache.org on 2012/12/17 10:52:46 UTC

svn commit: r1422820 - /camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java

Author: ningjiang
Date: Mon Dec 17 09:52:45 2012
New Revision: 1422820

URL: http://svn.apache.org/viewvc?rev=1422820&view=rev
Log:
Ignore the tests which are not supported after CAMEL-5871

Modified:
    camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java

Modified: camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java?rev=1422820&r1=1422819&r2=1422820&view=diff
==============================================================================
--- camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java (original)
+++ camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java Mon Dec 17 09:52:45 2012
@@ -86,7 +86,7 @@ public class BindyUnmarshalCommaIssueTes
     }
 
     @Test
-    //@Ignore
+    @Ignore("To fix CAMEL-5871. doesn't support the signle quote test case any more")
     public void testBindyUnmarshalSingleQuoteCommaIssueTwo() throws Exception {
         MockEndpoint mock = getMockEndpoint("mock:result");
         mock.expectedMessageCount(1);



Re: svn commit: r1422820 -/camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java

Posted by Willem jiang <wi...@gmail.com>.
I just added a note to the Camel 2.11 release note.  

--  
Willem Jiang

Red Hat, Inc.
FuseSource is now part of Red Hat
Web: http://www.fusesource.com | http://www.redhat.com
Blog: http://willemjiang.blogspot.com (http://willemjiang.blogspot.com/) (English)
          http://jnn.iteye.com (http://jnn.javaeye.com/) (Chinese)
Twitter: willemjiang  
Weibo: 姜宁willem





On Wednesday, December 19, 2012 at 6:00 PM, Claus Ibsen wrote:

> Hi Willem
>  
> We should add a note to release notes in Camel 2.11 that there is a
> change in camel-bindy that is not backwards compatible.
>  
>  
>  
>  
> On Mon, Dec 17, 2012 at 10:52 AM, <ningjiang@apache.org (mailto:ningjiang@apache.org)> wrote:
> > Author: ningjiang
> > Date: Mon Dec 17 09:52:45 2012
> > New Revision: 1422820
> >  
> > URL: http://svn.apache.org/viewvc?rev=1422820&view=rev
> > Log:
> > Ignore the tests which are not supported after CAMEL-5871
> >  
> > Modified:
> > camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java
> >  
> > Modified: camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java
> > URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java?rev=1422820&r1=1422819&r2=1422820&view=diff
> > ==============================================================================
> > --- camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java (original)
> > +++ camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java Mon Dec 17 09:52:45 2012
> > @@ -86,7 +86,7 @@ public class BindyUnmarshalCommaIssueTes
> > }
> >  
> > @Test
> > - //@Ignore
> > + @Ignore("To fix CAMEL-5871. doesn't support the signle quote test case any more")
> > public void testBindyUnmarshalSingleQuoteCommaIssueTwo() throws Exception {
> > MockEndpoint mock = getMockEndpoint("mock:result");
> > mock.expectedMessageCount(1);
>  
>  
>  
>  
>  
> --  
> Claus Ibsen
> -----------------
> Red Hat, Inc.
> FuseSource is now part of Red Hat
> Email: cibsen@redhat.com (mailto:cibsen@redhat.com)
> Web: http://fusesource.com
> Twitter: davsclaus
> Blog: http://davsclaus.com
> Author of Camel in Action: http://www.manning.com/ibsen




Re: svn commit: r1422820 - /camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java

Posted by Claus Ibsen <cl...@gmail.com>.
Hi Willem

We should add a note to release notes in Camel 2.11 that there is a
change in camel-bindy that is not backwards compatible.




On Mon, Dec 17, 2012 at 10:52 AM,  <ni...@apache.org> wrote:
> Author: ningjiang
> Date: Mon Dec 17 09:52:45 2012
> New Revision: 1422820
>
> URL: http://svn.apache.org/viewvc?rev=1422820&view=rev
> Log:
> Ignore the tests which are not supported after CAMEL-5871
>
> Modified:
>     camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java
>
> Modified: camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java
> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java?rev=1422820&r1=1422819&r2=1422820&view=diff
> ==============================================================================
> --- camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java (original)
> +++ camel/trunk/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv2/BindyUnmarshalCommaIssueTest.java Mon Dec 17 09:52:45 2012
> @@ -86,7 +86,7 @@ public class BindyUnmarshalCommaIssueTes
>      }
>
>      @Test
> -    //@Ignore
> +    @Ignore("To fix CAMEL-5871. doesn't support the signle quote test case any more")
>      public void testBindyUnmarshalSingleQuoteCommaIssueTwo() throws Exception {
>          MockEndpoint mock = getMockEndpoint("mock:result");
>          mock.expectedMessageCount(1);
>
>



-- 
Claus Ibsen
-----------------
Red Hat, Inc.
FuseSource is now part of Red Hat
Email: cibsen@redhat.com
Web: http://fusesource.com
Twitter: davsclaus
Blog: http://davsclaus.com
Author of Camel in Action: http://www.manning.com/ibsen