K2-NamespaceURIForPrefixFunc-2 test fails

[Sorin Marian Nasoi]

this is the ctest output:

test w3c_testsuite/XQuery/Functions/QNameFunc/NamespaceURIForPrefixFunc/K2-NamespaceURIForPrefixFunc-2
=== Query: ===
(:*******************************************************:)
(: Test: K2-NamespaceURIForPrefixFunc-2 :)
(: Written by: Frans Englich :)
(: Date: 2007-11-22T11:31:21+01:00 :)
(: Purpose: Test that the correct default namespace is picked up(#2). :)
(:*******************************************************:)
namespace-uri-for-prefix("", exactly-one(<e xmlns="http://www.example.com/A" xmlns:A="http://www.example.com/C">
              <b xmlns:B="http://www.example.com/C" xmlns=""/>
      </e>/b)) eq ""
=== end of Query ===
=== Result: ===
true
=== end of result ===
testdriver: non-canonical result for reference result # 1 doesn't match.

Actual and Reference canonical results are not identical

Actual Canonical Result:

<root>true</root>

Reference Canonical Result:

<root></root>

See line 1, col 6 of expected result.
Actual: <<root>true</root>>
Expected: <<root></root>>
testdriver: canonical result for reference result # 1 doesn't match.
testdriver: none of the reference results matched

[Sorin Marian Nasoi (sorin.marian.nasoi)]

In XQTS_1_0_3 the expected result for this test is 'true'.
According to the:
- W3C F&O Spec: http://www.w3.org/TR/xpath-functions-30/#func-namespace-uri-for-prefix
- decisions taken in bug 11590: http://www.w3.org/Bugs/Public/show_bug.cgi?id=11590

The correct result in this case is the empty sequence: because () eq "" returns ()

Because of the fact that at this moment the RQ uses XQTS_1_0_3, the test test/rbkt/w3c_testsuite/XQuery/Functions/QNameFunc/NamespaceURIForPrefixFunc/K2-NamespaceURIForPrefixFunc-2
is marked as an expected failure in
test/tbkt/Queries/CMakeLists.txt.
Also the test was manually added in the test/tbkt/Queries/w3c_known_failures.txt in order for the change to go through the RQ.

The proposed changes for this bug can be found on this public branch: lp:~zorba-coders/zorba/bug_872732

[Matthias Brantner (matthias-brantner)]

Attached, please find a patch that simplifies the function a bit (apply using bzr merge name.diff).

[Sorin Marian Nasoi (sorin.marian.nasoi)]

Fixed by commit #10524.