19881 2012-11-06 15:05:19 +0000 add a <dependency> node to the tests that use HOF feature 2013-01-02 14:23:53 +0000 1 1 1 Unclassified XPath / XQuery / XSLT XQuery 3 & XPath 3 Test Suite Working drafts PC Linux RESOLVED FIXED P2 normal --- 1 spungi oneil g mike public-qt-comments oldest_to_newest 77948 0 spungi 2012-11-06 15:05:19 +0000 The HOF feature is optional, so the tests in FOTS using the HOF feature should have a <dependency type="feature" value="higher-order-functions" satisfied="true"/> node added. If I am not missing something, all test cases that satisfy the condition: matches(fn:data($test-case/fots:test),"(function\-lookup)|(function\-name)|(function\-arity)|(map)|(filter)|(fold\-left)|(fold\-right)|(map\-pairs)|(#[1-9])")) should be marked with a HOF dependency. 77951 1 mike 2012-11-06 15:37:31 +0000 I have marked a great many tests with the dependency higherOrderFunctions; see for example http://dev.w3.org/cvsweb/2011/QT3-test-suite/fn/function-lookup.xml?rev=1.20;content-type=text%2Fplain where it is defined at test-set level. If you find specific tests that are missing this dependency - and there are bound to be some - please let us know. 78012 2 1246 spungi 2012-11-07 21:59:57 +0000 Created attachment 1246 These tests do not have a HOF dependency defined on neither test case nor test set level If I am not missing something, all the tests with a $test node that returns true for: matches($test,"(function\-lookup)|(function\-name)|(function\-arity)|(map)|(filter)|(fold\-left)|(fold\-right)|(map\-pairs)|((a-z)*#[0-9])")) should be marked with a HOF dependency at the test case level. The mentioned tests are checked for a HOF dependency at both test case and test set level first. 78036 3 oneil 2012-11-08 10:00:59 +0000 Thanks Sorin for the comprehensive list. I have made the changes and committed them to cvs. No change were made to the test-cases: require-higher-order-function-4-ns require-higher-order-function-9-ns prohibit-higher-order-function-4-ns According to the description of these the desired affect of switching off the HOF features in the implementation is that we expect an error. Just to note that some implementations, like ours, may not be able to switch of features externally! I think Ghislain should look at these test cases again. 78067 4 spungi 2012-11-08 18:00:15 +0000 Are you sure you pushed the updates to CVS for all reported cases? The following have not yet been updated: <test-set name="fn-data"> <test-case name="K2-DataFunc-5"/> </test-set> <test-set name="fn-map"> <test-case name="map-006"/> </test-set> Also please wait a bit before marking the bug as fixed because I will post another list of test-cases that might need HOF dependency added. The REGEX I used to identify the first list of tests did not catch all concurrences. 78071 5 spungi 2012-11-08 18:30:36 +0000 (In reply to comment #4) > Are you sure you pushed the updates to CVS for all reported cases? > > The following have not yet been updated: > <test-set name="fn-data"> > <test-case name="K2-DataFunc-5"/> > </test-set> > <test-set name="fn-map"> > <test-case name="map-006"/> > </test-set> > > Also please wait a bit before marking the bug as fixed because I will post > another list of test-cases that might need HOF dependency added. > > The REGEX I used to identify the first list of tests did not catch all > concurrences. OK: here are all the cases that should be looked into: <test set name="fn-map">add HOF dependency for the whole test set</test set> <test set name="prod-NamedFunctionRef">add HOF dependency for the whole test set</test set> <test-set name="fn-data"> <test-case name="K2-DataFunc-5"/> </test-set> <test-set name="fn-map"> <test-case name="map-006"/> </test-set> 80350 6 g 2012-12-18 14:01:30 +0000 Hi O'Neil, You are right - the test cases *-ns correspond to the assumption that a given feature (here, HOF) is not supported, and are hence marked with <dependency type="feature" value="higherOrderFunctions" satisfied="false"/>. If an implementation does not support HOF, it must run these tests to check that it behaves appropriately for require-feature/prohibit-feature. (In reply to comment #3) > Thanks Sorin for the comprehensive list. I have made the changes and > committed them to cvs. > > No change were made to the test-cases: > require-higher-order-function-4-ns > require-higher-order-function-9-ns > prohibit-higher-order-function-4-ns > > According to the description of these the desired affect of switching off > the HOF features in the implementation is that we expect an error. > > Just to note that some implementations, like ours, may not be able to switch > of features externally! I think Ghislain should look at these test cases > again. 80351 7 g 2012-12-18 14:02:59 +0000 I added HOF dependency tags to the following test sets: fn-map prod-namedFunctionRef as well as to the following test cases: fn-unparsed-text-003 fn-unparsed-text-004 fn-unparsed-text-available-003 fn-unparsed-text-available-004 fn-unparsed-text-lines-003 fn-unparsed-text-lines-004 K2-DataFunc-5 80352 8 g 2012-12-18 14:07:49 +0000 > Just to note that some implementations, like ours, may not be able to switch > of features externally! I think Ghislain should look at these test cases > again. My understanding is that it is not required to switch off HOF. If an implementation supports HOF and does not provide a means of switching it off externally, I think the semantics of the tags set to false is that it should just skip (ignore) these tests. 80406 9 g 2012-12-19 14:00:31 +0000 I have also added a dependency tag on test case contextDecl-055. 1246 2012-11-07 21:59:57 +0000 2012-11-07 21:59:57 +0000 These tests do not have a HOF dependency defined on neither test case nor test set level tests.xml application/xml 4091 spungi PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iVVRGLTgiPz4KPHRlc3RzPgogIDx0ZXN0LXNl dCBuYW1lPSJmbi1hdmFpbGFibGUtZW52aXJvbm1lbnQtdmFyaWFibGVzIj4KICAgIDx0ZXN0LWNh c2UgbmFtZT0iZm4tYXZhaWxhYmxlLWVudmlyb25tZW50LXZhcmlhYmxlcy0wMDEiLz4KICAgIDx0 ZXN0LWNhc2UgbmFtZT0iZm4tYXZhaWxhYmxlLWVudmlyb25tZW50LXZhcmlhYmxlcy0wMDIiLz4K ICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tYXZhaWxhYmxlLWVudmlyb25tZW50LXZhcmlhYmxlcy0w MDMiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tYXZhaWxhYmxlLWVudmlyb25tZW50LXZhcmlh Ymxlcy0wMDQiLz4KICA8L3Rlc3Qtc2V0PgogIDx0ZXN0LXNldCBuYW1lPSJmbi1kYXRhIj4KICAg IDx0ZXN0LWNhc2UgbmFtZT0iSzItRGF0YUZ1bmMtNSIvPgogIDwvdGVzdC1zZXQ+CiAgPHRlc3Qt c2V0IG5hbWU9ImZuLWVudmlyb25tZW50LXZhcmlhYmxlIj4KICAgIDx0ZXN0LWNhc2UgbmFtZT0i ZW52aXJvbm1lbnQtdmFyaWFibGUtMDAxIi8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImVudmlyb25t ZW50LXZhcmlhYmxlLTAwMiIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJlbnZpcm9ubWVudC12YXJp YWJsZS0wMDMiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZW52aXJvbm1lbnQtdmFyaWFibGUtMDA0 Ii8+CiAgPC90ZXN0LXNldD4KICA8dGVzdC1zZXQgbmFtZT0iZm4taGFzLWNoaWxkcmVuIj4KICAg IDx0ZXN0LWNhc2UgbmFtZT0iZm4taGFzLWNoaWxkcmVuLTAwMSIvPgogICAgPHRlc3QtY2FzZSBu YW1lPSJmbi1oYXMtY2hpbGRyZW4tMDAyIi8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLWhhcy1j aGlsZHJlbi0wMDMiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4taGFzLWNoaWxkcmVuLTAwNCIv PgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1oYXMtY2hpbGRyZW4tMDA4Ii8+CiAgICA8dGVzdC1j YXNlIG5hbWU9ImZuLWhhcy1jaGlsZHJlbi0wMTAiLz4KICA8L3Rlc3Qtc2V0PgogIDx0ZXN0LXNl dCBuYW1lPSJmbi1pbm5lcm1vc3QiPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1pbm5lcm1vc3Qt MDAyIi8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLWlubmVybW9zdC0wMDQiLz4KICAgIDx0ZXN0 LWNhc2UgbmFtZT0iZm4taW5uZXJtb3N0LTAwNSIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1p bm5lcm1vc3QtMDA3Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLWlubmVybW9zdC0wMTEiLz4K ICA8L3Rlc3Qtc2V0PgogIDx0ZXN0LXNldCBuYW1lPSJmbi1tYXAiPgogICAgPHRlc3QtY2FzZSBu YW1lPSJtYXAtMDA2Ii8+CiAgPC90ZXN0LXNldD4KICA8dGVzdC1zZXQgbmFtZT0iZm4tbWFwLXBh aXJzIj4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJzLTAwMSIvPgogICAgPHRlc3Qt Y2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDAyIi8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLW1h cC1wYWlycy0wMDMiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJzLTAwNCIvPgog ICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDA1Ii8+CiAgICA8dGVzdC1jYXNlIG5h bWU9ImZuLW1hcC1wYWlycy0wMDYiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJz LTAwNyIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDA4Ii8+CiAgICA8dGVz dC1jYXNlIG5hbWU9ImZuLW1hcC1wYWlycy0wMDkiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4t bWFwLXBhaXJzLTAxMSIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDEyIi8+ CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLW1hcC1wYWlycy0wMTMiLz4KICAgIDx0ZXN0LWNhc2Ug bmFtZT0iZm4tbWFwLXBhaXJzLTAxNCIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFp cnMtMDE1Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLW1hcC1wYWlycy0wMTYiLz4KICAgIDx0 ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJzLTAxNyIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJm bi1tYXAtcGFpcnMtMDE4Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLW1hcC1wYWlycy0wMTki Lz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJzLTAyMCIvPgogICAgPHRlc3QtY2Fz ZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDIxIi8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLW1hcC1w YWlycy0wMjIiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJzLTAyMyIvPgogICAg PHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDI0Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9 ImZuLW1hcC1wYWlycy0wMjUiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFwLXBhaXJzLTAy NiIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDI3Ii8+CiAgICA8dGVzdC1j YXNlIG5hbWU9ImZuLW1hcC1wYWlycy0wMjgiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iZm4tbWFw LXBhaXJzLTAyOSIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1tYXAtcGFpcnMtMDMwIi8+CiAg PC90ZXN0LXNldD4KICA8dGVzdC1zZXQgbmFtZT0iZm4tb3V0ZXJtb3N0Ij4KICAgIDx0ZXN0LWNh c2UgbmFtZT0iZm4tb3V0ZXJtb3N0LTAwMiIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJmbi1vdXRl cm1vc3QtMDA0Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9ImZuLW91dGVybW9zdC0wMDUiLz4KICAg IDx0ZXN0LWNhc2UgbmFtZT0iZm4tb3V0ZXJtb3N0LTAwNyIvPgogICAgPHRlc3QtY2FzZSBuYW1l PSJmbi1vdXRlcm1vc3QtMDExIi8+CiAgPC90ZXN0LXNldD4KICA8dGVzdC1zZXQgbmFtZT0ib3At Y29uY2F0Ij4KICAgIDx0ZXN0LWNhc2UgbmFtZT0ib3AtY29uY2F0LTE4Ii8+CiAgPC90ZXN0LXNl dD4KICA8dGVzdC1zZXQgbmFtZT0icHJvZC1DYXN0RXhwci5zY2hlbWEiPgogICAgPHRlc3QtY2Fz ZSBuYW1lPSJDYXN0QXMtVW5pb25UeXBlLTciLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iQ2FzdEFz LVVuaW9uVHlwZS04Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9IkNhc3RBcy1VbmlvblR5cGUtMTMi Lz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iQ2FzdEFzLVVuaW9uVHlwZS0xNCIvPgogICAgPHRlc3Qt Y2FzZSBuYW1lPSJDYXN0QXMtVW5pb25UeXBlLTE4Ii8+CiAgICA8dGVzdC1jYXNlIG5hbWU9IkNh c3RBcy1VbmlvblR5cGUtMjQiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iQ2FzdEFzLVVuaW9uVHlw ZS0yNiIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJDYXN0QXMtVW5pb25UeXBlLTMwIi8+CiAgICA8 dGVzdC1jYXNlIG5hbWU9IkNhc3RBcy1VbmlvblR5cGUtMzEiLz4KICAgIDx0ZXN0LWNhc2UgbmFt ZT0iQ2FzdEFzLVVuaW9uVHlwZS0zMiIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJDYXN0QXMtTGlz dFR5cGUtOSIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJDYXN0QXMtTGlzdFR5cGUtMTAiLz4KICAg IDx0ZXN0LWNhc2UgbmFtZT0iQ2FzdEFzLUxpc3RUeXBlLTE4Ii8+CiAgICA8dGVzdC1jYXNlIG5h bWU9IkNhc3RBcy1MaXN0VHlwZS0xOSIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJDYXN0QXMtTGlz dFR5cGUtMjQiLz4KICAgIDx0ZXN0LWNhc2UgbmFtZT0iQ2FzdEFzLUxpc3RUeXBlLTI2Ii8+CiAg ICA8dGVzdC1jYXNlIG5hbWU9IkNhc3RBcy1MaXN0VHlwZS0yNyIvPgogIDwvdGVzdC1zZXQ+CiAg PHRlc3Qtc2V0IG5hbWU9InByb2QtTW9kdWxlSW1wb3J0Ij4KICAgIDx0ZXN0LWNhc2UgbmFtZT0i bW9kdWxlcy1wdWItcHJpdi0zNyIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJtb2R1bGVzLXB1Yi1w cml2LTM4Ii8+CiAgPC90ZXN0LXNldD4KICA8dGVzdC1zZXQgbmFtZT0icHJvZC1SZXF1aXJlUHJv aGliaXRGZWF0dXJlIj4KICAgIDx0ZXN0LWNhc2UgbmFtZT0icmVxdWlyZS1oaWdoZXItb3JkZXIt ZnVuY3Rpb24tNC1ucyIvPgogICAgPHRlc3QtY2FzZSBuYW1lPSJyZXF1aXJlLWhpZ2hlci1vcmRl ci1mdW5jdGlvbi05LW5zIi8+CiAgICA8dGVzdC1jYXNlIG5hbWU9InByb2hpYml0LWhpZ2hlci1v cmRlci1mdW5jdGlvbi00LW5zIi8+CiAgPC90ZXN0LXNldD4KPC90ZXN0cz4=