<?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>27711</bug_id>
          
          <creation_ts>2014-12-28 10:17:26 +0000</creation_ts>
          <short_desc>modules-none</short_desc>
          <delta_ts>2015-02-10 11:48:08 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>XPath / XQuery / XSLT</product>
          <component>XQuery 3 &amp; XPath 3 Test Suite</component>
          <version>Working drafts</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</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="Benito van der Zander">benito</reporter>
          <assigned_to name="O&apos;Neil Delpratt">oneil</assigned_to>
          <cc>mike</cc>
          
          <qa_contact name="Mailing list for public feedback on specs from XSL and XML Query WGs">public-qt-comments</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>116700</commentid>
    <comment_count>0</comment_count>
    <who name="Benito van der Zander">benito</who>
    <bug_when>2014-12-28 10:17:26 +0000</bug_when>
    <thetext>The test-case modules-none has a &lt;module uri=&quot;http://www.w3.org/TestModules/none&quot; file=&quot;ModuleImport/empty-lib.xq&quot;/&gt; dependency, to a module that does not exist.

Now the test-case should check the behaviour when a not existing module is encountered, but I do not think the not existing file of the not existing module should be mentioned in a &lt;module/&gt; element, since it is the test driver loading the modules of &lt;module/&gt; elements, not the test-case. And there is no point in testing if the test driver can handle not existing files.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>116701</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2014-12-28 17:28:51 +0000</bug_when>
    <thetext>I would have some sympathy with the argument were it not that the test was written 9 years ago, and therefore presumably many people have written test drivers that can handle it, and these test drivers might well have to change if it were done differently. Yes, the task of resolving module URIs is to some extent a matter for test drivers, and therefore handling of an absent module URI is, to some extent, testing the test driver rather than the product; but I think a little bit of fuzziness at the edges is something we have to accept as a fact of life. This test has been around for a long time and I don&apos;t think there&apos;s sufficient justification here for changing it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>116703</commentid>
    <comment_count>2</comment_count>
    <who name="Benito van der Zander">benito</who>
    <bug_when>2014-12-28 20:26:31 +0000</bug_when>
    <thetext>&gt; were it not that the test was written 9 years ago,


The test is so old, but the meta data about the module not.

That was changed for all tests with the new XQTS.

Back then the file was just mentioned in a global map that was not really linked to the test except for the description. The new metadata seems to say that a module with that namespace from that file is used in the test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>116754</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Kay">mike</who>
    <bug_when>2015-01-03 20:25:35 +0000</bug_when>
    <thetext>I vote for rejecting this bug. It&apos;s perfectly reasonable to have a test that attempts to import a non-existant module, and I can&apos;t think of any other way of writing the test. If it&apos;s the test driver that handles the error rather than the product-under-test, so be it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>116757</commentid>
    <comment_count>4</comment_count>
    <who name="Benito van der Zander">benito</who>
    <bug_when>2015-01-03 20:45:33 +0000</bug_when>
    <thetext>
&gt;and I can&apos;t think of any other way of writing the test.

Remove &lt;module uri=&quot;http://www.w3.org/TestModules/none&quot; file=&quot;ModuleImport/empty-lib.xq&quot;/&gt; and keep the test itself unchanged?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>117802</commentid>
    <comment_count>5</comment_count>
    <who name="O&apos;Neil Delpratt">oneil</who>
    <bug_when>2015-02-10 11:47:45 +0000</bug_when>
    <thetext>The WG has decided to remove the test in question.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>