[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Handle special characters in Junit report

From: Bhuvaneswaran A <bhuvan_at_collab.net>
Date: Fri, 04 Dec 2009 20:14:22 +0530

On Fri, 2009-12-04 at 15:29 +0100, Branko Čibej wrote:
> Bhuvaneswaran A wrote:
> > On Fri, 2009-12-04 at 12:36 +0000, Julian Foad wrote:
> >
> >> Bhuvaneswaran A wrote:
> >>
> >>> Please find attached the revised patch. I incorporated following
> >>> feedback:
> >>> a) Fix the array slicing part
> >>> b) Escape using ord() instead of removing those characters
> >>> c) Handle "]]>" in CDATA section
> >>> d) Define the ascii table globally (once) and re-use
> >>>
> >>> I also verified this fix by generating the junit files for tests having
> >>> special characters and simulating a test that has "]]>" in failure text.
> >>> With this patch, it generates valid junit file.
> >>>
> >> It looks great. You could also move the definition of 'chars_to_remove'
> >> out of the function, but either way it's fine. Go on, commit it!
> >>
> >
> > Branko, Julian: Thank you for the review comments.
> >
> > Incorporated the above suggestion and committed in r887178.
>
> Wait, you committed a script that does URL quoting on XML contents? Did
> you look at the output?

Branko, yes. The data goes in CDATA section in xml. With this fix, a
valid xml file is generated even if the failure message contain a)
control characters. b) contains "]]>" pattern.

-- 
Bhuvaneswaran A    
CollabNet Software P Ltd.  |  www.collab.net

Received on 2009-12-04 15:46:06 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.