-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Discrepancies between the QT4 tests and the Working Draft QT4 XPath spec. #82
Comments
Thanks. We've resolved that in future, tests created in anticipation of a change to the spec will be submitted as a PR and only merged in when the spec change is made. Meanwhile there will be cases where the tests are ahead of the spec or vice versa. Similarly, we're hoping to get the changes.xml mechanism into better shape. |
@johnlumley Thanks as well; I have moved those tests out of the test suite into a PR yesterday (#81). Feel free to keep us updated about other inconsistencies. |
I may as well use this issue to record other inconsistencies I find in the test suites. Here goes: Missing XQuery dependenciesThe following 7 (seven) test cases in test set (Corrected from original: |
Additional missing XQuery dependency declarations are on the following 7 test cases: |
@johnlumley Do you want me to add the dependencies, or are you already working on it? |
@ChristianGruen - I'd prefer you to manage the dependencies - my GIT-experience is limited and I fear mistakes! |
I’ve just added the dependency declarations; I hope I haven’t overlooked anything. |
Just checking some more from my last test run! More missing XQuery dependency declarations are on the following 12 test cases: ( |
I’ve fixed the two.
@michaelhkay may need to judge how to tackle those ones (they are generated). |
Sorry to drip feed. A few more: |
|
Fixed. In addition, |
Apart from the error caused by the generated tests, all other test cases should haven been fixed. |
@johnlumley Maybe we can close this issue and look at the generated tests in a separate issue. What do you think? |
Additional Missing XQuery dependenciesMore missing XQuery dependency declarations in the test suite as at 2024sep14 are on the following 13 test cases:
|
Error in
|
In working on testing a generator of an iXML grammar for XPath 4, I have uncovered a few cases where the test suite contains tests for features that have been proposed but do not yet appear in the working draft of the spec. This issue is to record what these are:
for key
&for value
iteration across maps.The test set
ForClause.map.xml
contains tests for the proposed iteration constructs across maps Extend FLWOR expressions to maps, where the proposed alterations to the XPath EBNF (and indeed description) do not appear in the current working draft XPath spec. Moreover the additional tests are not mentioned in the currentchanges.xml
record, though the addition of tests forfor member
array iteration is.The text was updated successfully, but these errors were encountered: