This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 21838 - [XQ3TS] require-feature-list-1-ns-1, 2, require-feature-list-1-s
Summary: [XQ3TS] require-feature-list-1-ns-1, 2, require-feature-list-1-s
Status: RESOLVED FIXED
Alias: None
Product: XPath / XQuery / XSLT
Classification: Unclassified
Component: XQuery 3 & XPath 3 Test Suite (show other bugs)
Version: Candidate Recommendation
Hardware: PC Windows NT
: P2 normal
Target Milestone: ---
Assignee: Ghislain Fourny
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-26 09:06 UTC by Tim Mills
Modified: 2013-05-07 16:17 UTC (History)
1 user (show)

See Also:


Attachments

Description Tim Mills 2013-04-26 09:06:03 UTC
These tests import a module which is missing from the environment.
Comment 1 O'Neil Delpratt 2013-05-02 09:41:57 UTC
I think this might be the intent of the test case, but I am asking Ghislain to comment here.

Please observe:

<dependency type="feature" value="higherOrderFunctions" satisfied="false"/>

This in affect switches off the HOF feature in the implementation so any call to require-feature on HOF should throw an error in the test case. However, I am still uncomfortable with this, I think we are over using the framework of the test suite somewhat. In our implementation we are not even running this test because the satisfied=false on the dependency.
Comment 2 Tim Mills 2013-05-02 09:45:39 UTC
(In reply to comment #1)

> This in affect switches off the HOF feature in the implementation so any
> call to require-feature on HOF should throw an error in the test case.

I'm fairly sure that this isn't the expected behaviour.  If the implementation can enable a feature in response to a require-feature, it is free to do so, and only raise an error if it can't.
Comment 3 Ghislain Fourny 2013-05-02 10:13:11 UTC
Good catch, I added the module dependency, awaiting your confirmation that it works for you.

I agree with Tim's comment 2, i.e., the WG made an explicit decision that it is okay for an implementation to activate a feature even if it had previously deactivated it following the dependency of false on that feature.

It comes down to saying that a dependency of false does not mean that the feature must be "not supported" for the test to be run, but that it must be "not activated" by default for the test to be run.

It is fine for an implementation that does support a feature to not run tests with a dependency of false on that feature, if it is unable to deactivate it, so I also agree with O'Neil's last sentence in comment 1.

Does it make sense?
Comment 4 O'Neil Delpratt 2013-05-07 16:17:42 UTC
This makes sense. I confirm tests have been updated.