gimsieke/xspec

One less XSpec namespace in reported differences

Closed this issue · 4 comments

At present, the markup in the 'Result' and 'Expecting' fragments always
includes two namespace declarations for XSpec.

This patch changes all s:* to x:* so there's only one namespace declaration
for XSpec in each fragment.

I haven't tried looking to find where to add 'exclude-namespace-prefixes'
to get rid of that one.  (And, of course, all of our tests always pass, so
we never need to see it anyway.)

Original issue reported on code.google.com by dev.xspec@menteithconsulting.com on 2 Apr 2009 at 8:42

Attachments:

The enclosed diff has been applied to the repository (I don't know when nor by 
who).

Still need to look at the last paragraph of the above description.

Original comment by fgeorges on 25 Jan 2010 at 4:07

  • Changed state: Started
Tony, could you upload a simple repro with a simple XSpec suite, the 
corresponding 
compiled XSLT stylesheet and the XML report?  So we can point exactly within 
those files 
to see where and which namespaces should not be there.

Thanks!

Original comment by fgeorges on 25 Jan 2010 at 7:31

The problem was fixed in r45 and r46.  I no longer have any issue with 
superfluous
namespaces.

Original comment by dev.xspec@menteithconsulting.com on 25 Jan 2010 at 8:03

So I mark the issue as fixed, thanks!

Original comment by fgeorges on 25 Jan 2010 at 9:12

  • Changed state: Fixed