<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>2910</bug_id>
          
          <creation_ts>2006-02-22 00:02:05 +0000</creation_ts>
          <short_desc>external variable tests</short_desc>
          <delta_ts>2006-05-12 14:31:44 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>XML Query Test Suite</product>
          <component>XML Query Test Suite</component>
          <version>0.8.6</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="David Carlisle">davidc</reporter>
          <assigned_to name="Carmelo Montanez">carmelo</assigned_to>
          
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>8377</commentid>
    <comment_count>0</comment_count>
    <who name="David Carlisle">davidc</who>
    <bug_when>2006-02-22 00:02:05 +0000</bug_when>
    <thetext>The new tests for external variables take on a somewhat surprising format.

Rather than a catalog element giving the value (and optionally the type)
they require running a preliminary query, capturing the result (including xpath
type) and passing it in to the main query.

The only portable output format from Xquery is an XML document (serialised or as
an in-memory dom of some sort) and this up to now has been the only required
output form used by the test suite.

However to take a particular example, to pass
extvardeclwithouttype-10.xq
I first need to evaluate 
extvardeclwithouttypetobind-10.xq
and get the value 1 (fine) but if I grab the result of that as an XML document
(the only way I&apos;m set up to grab it currently) I basically get a text node
with the string &quot;1&quot; and so the main query fails to add 1+1.

The reasons that I currently fail this test seem to be only distantly related to
the ability or not to handle external variables. (If the catalog had simply
specified that I need to set the variable x to the integer 1, I&apos;d have no trouble).

This requirement to capture the typed value of atomic values from a Query is a
major change in the test suite. I&apos;m not sure currently how feasible this would
be  for me to do (I may just decide to not take these tests) but I thought I&apos;d
raise the issue to see if any other designs were or could be considered. 

Hopefully as you approach CR you&apos;ll get other submitted results from more
commercial vendors and whether or not I skip a bunch of tests will be less of an
issue, but if I&apos;m faced with re-writing large chunks of my test harness to
accomodate this test group, other implementors may face the similar issues...

David</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>8615</commentid>
    <comment_count>1</comment_count>
    <who name="David Carlisle">davidc</who>
    <bug_when>2006-03-07 13:57:51 +0000</bug_when>
    <thetext>Some discussion of this on the query-talk list. It seems that other implementers
are happy enough with this as it is, and I have a scheme now that seems to work OK, so if you choose to close this or mark it as invalid or wontfix 
I won&apos;t object. (Although a little more documentation of the range of possibilities open for implementing this in a test harness would not be unwelcome, so I&apos;ll leave this open for now)

David
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9055</commentid>
    <comment_count>2</comment_count>
    <who name="Carmelo Montanez">carmelo</who>
    <bug_when>2006-04-06 15:08:42 +0000</bug_when>
    <thetext>Hey David:

Thanks for this comment and the suggestions you present.  I will think
of extra verbiage to put in the guidelines to help implementors run the tests.
I will consult the task force about the catalog dictating a more direct way for the variable binding.  I will mark the bug as ssigned and then as wontfix.

Thanks,
Carmelo
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9766</commentid>
    <comment_count>3</comment_count>
    <who name="David Carlisle">davidc</who>
    <bug_when>2006-05-12 14:31:44 +0000</bug_when>
    <thetext>No objection, I flagged the issue but a) it wasn&apos;t as hard as I though for me to cope, and b) other implementors said they had no objections, so I&apos;m closing.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>