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 21238 - Tests involving schema imports: add XQST0009
Summary: Tests involving schema imports: add XQST0009
Status: CLOSED FIXED
Alias: None
Product: XPath / XQuery / XSLT
Classification: Unclassified
Component: XQuery 3 & XPath 3 Test Suite (show other bugs)
Version: Candidate Recommendation
Hardware: PC All
: P2 normal
Target Milestone: ---
Assignee: O'Neil Delpratt
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-10 12:28 UTC by Christian Gruen
Modified: 2013-06-07 11:25 UTC (History)
1 user (show)

See Also:


Attachments
Patch with schema dependencies (6.22 KB, text/plain)
2013-04-30 09:05 UTC, Christian Gruen
Details
Patch with schema dependencies (updated) (6.48 KB, text/plain)
2013-04-30 11:38 UTC, Christian Gruen
Details
Schema dependencies added to fn:nilled() tests (1.33 KB, patch)
2013-05-06 15:02 UTC, Christian Gruen
Details

Description Christian Gruen 2013-03-10 12:28:34 UTC
As the following QT3TS tests (and maybe some more) include schema imports, I would expect error XQST0009 as valid alternative:

- ST-Data001
- CastAsNamespaceSensitiveType-6
- CastAsNamespaceSensitiveType-7
- CastAsNamespaceSensitiveType-8
- CastAsNamespaceSensitiveType-9
- CastAsNamespaceSensitiveType-10
- CastAsNamespaceSensitiveType-11
- CastAsNamespaceSensitiveType-12
- nscons-040
- nscons-041
- Constr-cont-nsmode-11
- instanceof112
- instanceof113
- instanceof114
- instanceof115
- instanceof116
- instanceof117
- instanceof118
- instanceof119
- instanceof120
- instanceof135
- instanceof136
- instanceof137
- instanceof138
- instanceof139
- instanceof140
- instanceof141
- cbcl-module-001
- require-schema-aware-1-s
- require-schema-aware-1-ns
- require-schema-aware-2-s
- require-schema-aware-2-ns
- require-schema-aware-3-s
- require-all-optional-features-1-s
- require-all-optional-features-1-ns1
- require-all-optional-features-1-ns2
- require-all-optional-features-1-ns3
- require-all-optional-features-1-ns4
- prohibit-all-optional-features-2-s
- prohibit-all-optional-features-2-ns
- prohibit-all-optional-features-3-s
- prohibit-all-optional-features-3-ns
- typeswitch-112
- typeswitch-113
- typeswitch-114
- typeswitch-115
- typeswitch-116
- extvardef-023
- extvardef-024
- extvardef-025
- extvardef-026
- extvardef-027
- FOTY0012
- XQDY0027
- XQDY0084
- XQST0036_1
- XQST0036_2
- XQST0036_3
- XQST0036_4
- XQST0036_5
- XQST0036_6
- XQST0036_7
- XQST0058
- XQST0059_1
- XQST0059_3
- XQST0059_6
Comment 1 Michael Kay 2013-03-10 18:16:24 UTC
The general policy is not to provide an alternative result for such tests, but rather to document the dependency of the test on optional features, so that a product which does not support the feature does not run the test. In many of these cases (those I checked) there is no explicit dependency on schema-awareness (arguably there should be), but there is an implicit dependency in that the environment contains a <schema> element.
Comment 2 O'Neil Delpratt 2013-03-20 12:17:21 UTC
(In reply to comment #1)

I believe we can mark this bug as resolved.
Comment 3 Christian Gruen 2013-03-20 12:25:19 UTC
Agreed.. Thanks.
Comment 4 Christian Gruen 2013-03-29 09:32:44 UTC
I have reopened this issue because a large number of test cases (appr. 6000, or 23%) seem to have an implicit implicit schema dependency. While we could check the necessity of all schema imports for those tests, I believe it is less effort if we explicitly add a schema dependency to the tests that I’ve initially listed.
Comment 5 Christian Gruen 2013-04-30 09:05:47 UTC
Created attachment 1351 [details]
Patch with schema dependencies

I have added a patch that includes all schema dependencies that I believe are missing in the current test suite.
Comment 6 Christian Gruen 2013-04-30 11:38:04 UTC
Created attachment 1352 [details]
Patch with schema dependencies (updated)

The updated patch includes an additional schema-import dependency for the itunes query.
Comment 7 O'Neil Delpratt 2013-04-30 13:03:59 UTC
Thanks Christian for the patch. I will apply it
Comment 8 Christian Gruen 2013-05-01 07:56:35 UTC
Thanks O'Neil! I noticed two more test cases need to be added: "orderBy29" and "orderBy52" sort their XML input based on a given schema. Feel free to tell me if you need an updated patch, or changes to the given patches.
Comment 9 O'Neil Delpratt 2013-05-01 11:06:00 UTC
Patch applied and updated to cvs. I will add the schemaImport dependency to the test cases mentioned in comment #8
Comment 10 O'Neil Delpratt 2013-05-01 12:33:44 UTC
Bug fixes complete and committed to subversion
Comment 11 Christian Gruen 2013-05-06 15:02:37 UTC
Created attachment 1357 [details]
Schema dependencies added to fn:nilled() tests

I have stumbled across some more test cases with unspecified schema dependencies; please have a look at my new patch.
Comment 12 Christian Gruen 2013-05-14 17:41:06 UTC
I have closed the issues #21937 and #21723, and I am adding five test cases to this issue, which I believe require yet another schema import dependency.

- WindowingUseCase07S
- WindowingUseCase11S
- WindowingUseCase13S
- LocalNameFromQNameFunc021
- NamespaceURIFromQNameFunc021

As mentioned in #21937, I’ll be glad to help out (e.g. by updating the patch or directly commit the changes by myself).
Comment 13 Christian Gruen 2013-06-05 09:44:06 UTC
Sorry for my persistence; has there been some discussion on this?
Comment 14 O'Neil Delpratt 2013-06-05 10:59:43 UTC
thanks for the reminder. I have committed the bug fixes to cvs. I confirm that I have addressed the issues raised in comment #11 and #12
Comment 15 Christian Gruen 2013-06-05 11:48:24 UTC
Thanks, O’Neil. I can confirm that all test cases are fixed. I’m embarrassed to notice, however, that I have forgotten the two test cases that had been added by Michael in bug #21723:

- NamespaceURIFromQNameFunc021a
- LocalNameFromQNameFunc021a

They also require a schema import dependency, similar to the two original test cases.
Comment 16 O'Neil Delpratt 2013-06-07 11:01:45 UTC
(In reply to comment #15)
dependency on schema import added
Comment 17 Christian Gruen 2013-06-07 11:25:46 UTC
Thanks; closed.