You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-dev@db.apache.org by "Michael Watzek (JIRA)" <ji...@apache.org> on 2005/10/04 12:56:47 UTC
[jira] Created: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
Implement new JDO 2 query tests cases concerning keywords.
----------------------------------------------------------
Key: JDO-157
URL: http://issues.apache.org/jira/browse/JDO-157
Project: JDO
Type: New Feature
Components: tck20
Reporter: Michael Watzek
Assigned to: Michael Watzek
We need 4 new test classes, one for each of the following assertions:
- A14.4-6: Keywords must not be used as package names, class names, parameter names, or variable names in queries.
- A14.4-7: Keywords are permitted as field names only if they are on the right side of the "." in field access expressions.
- A14.6.13-1: The String version of Query represents all query elements using a single string. The string contains the following structure.
- A14.6.13-2: Keywords, identified above in bold, are either all upper-case or all lower-case. Keywords cannot be mixed case.
Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#Keywords.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Updated: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
Posted by "Michael Watzek (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/jira/browse/JDO-157?page=all ]
Michael Watzek updated JDO-157:
-------------------------------
Attachment: JDO-157.patch2
The second patch contains all the comments.
> Implement new JDO 2 query tests cases concerning keywords.
> ----------------------------------------------------------
>
> Key: JDO-157
> URL: http://issues.apache.org/jira/browse/JDO-157
> Project: JDO
> Type: New Feature
> Components: tck20
> Reporter: Michael Watzek
> Assignee: Michael Watzek
> Attachments: JDO-157.patch, JDO-157.patch2
>
> We need 4 new test classes, one for each of the following assertions:
> - A14.4-6: Keywords must not be used as package names, class names, parameter names, or variable names in queries.
> - A14.4-7: Keywords are permitted as field names only if they are on the right side of the "." in field access expressions.
> - A14.6.13-1: The String version of Query represents all query elements using a single string. The string contains the following structure.
> - A14.6.13-2: Keywords, identified above in bold, are either all upper-case or all lower-case. Keywords cannot be mixed case.
> Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#Keywords.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Updated: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
Posted by "Michael Watzek (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/jira/browse/JDO-157?page=all ]
Michael Watzek updated JDO-157:
-------------------------------
Attachment: JDO-157.patch
This patch implements the assertions above. It depends on patches for JDO-156 and JDO-168, in that order. For this reason, those patches must be checked in first.
> Implement new JDO 2 query tests cases concerning keywords.
> ----------------------------------------------------------
>
> Key: JDO-157
> URL: http://issues.apache.org/jira/browse/JDO-157
> Project: JDO
> Type: New Feature
> Components: tck20
> Reporter: Michael Watzek
> Assignee: Michael Watzek
> Attachments: JDO-157.patch
>
> We need 4 new test classes, one for each of the following assertions:
> - A14.4-6: Keywords must not be used as package names, class names, parameter names, or variable names in queries.
> - A14.4-7: Keywords are permitted as field names only if they are on the right side of the "." in field access expressions.
> - A14.6.13-1: The String version of Query represents all query elements using a single string. The string contains the following structure.
> - A14.6.13-2: Keywords, identified above in bold, are either all upper-case or all lower-case. Keywords cannot be mixed case.
> Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#Keywords.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Resolved: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
Posted by "Michael Watzek (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/jira/browse/JDO-157?page=all ]
Michael Watzek resolved JDO-157:
--------------------------------
Resolution: Fixed
The second patch has been checked in.
> Implement new JDO 2 query tests cases concerning keywords.
> ----------------------------------------------------------
>
> Key: JDO-157
> URL: http://issues.apache.org/jira/browse/JDO-157
> Project: JDO
> Type: New Feature
> Components: tck20
> Reporter: Michael Watzek
> Assignee: Michael Watzek
> Attachments: JDO-157.patch, JDO-157.patch2
>
> We need 4 new test classes, one for each of the following assertions:
> - A14.4-6: Keywords must not be used as package names, class names, parameter names, or variable names in queries.
> - A14.4-7: Keywords are permitted as field names only if they are on the right side of the "." in field access expressions.
> - A14.6.13-1: The String version of Query represents all query elements using a single string. The string contains the following structure.
> - A14.6.13-2: Keywords, identified above in bold, are either all upper-case or all lower-case. Keywords cannot be mixed case.
> Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#Keywords.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
Posted by "Michael Bouschen (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/jira/browse/JDO-157?page=comments#action_12331882 ]
Michael Bouschen commented on JDO-157:
--------------------------------------
Just some minor comments:
Class CompanyModelReader
- The CompanyModelReader change is already checked in as part of patch 156.
Class FullName
- minor: you can shorten the implementation of equals:
public boolean equals(Object o) {
FullName other = (FullName) o;
if (this.firstName == null) {
return other.firstName == null;
}
if (this.lastName == null) {
return other.lastName == null;
}
...
- How about adding some class javadoc describing why you added two getter methods getFirstName and getFirstname etc.?
> Implement new JDO 2 query tests cases concerning keywords.
> ----------------------------------------------------------
>
> Key: JDO-157
> URL: http://issues.apache.org/jira/browse/JDO-157
> Project: JDO
> Type: New Feature
> Components: tck20
> Reporter: Michael Watzek
> Assignee: Michael Watzek
> Attachments: JDO-157.patch
>
> We need 4 new test classes, one for each of the following assertions:
> - A14.4-6: Keywords must not be used as package names, class names, parameter names, or variable names in queries.
> - A14.4-7: Keywords are permitted as field names only if they are on the right side of the "." in field access expressions.
> - A14.6.13-1: The String version of Query represents all query elements using a single string. The string contains the following structure.
> - A14.6.13-2: Keywords, identified above in bold, are either all upper-case or all lower-case. Keywords cannot be mixed case.
> Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#Keywords.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
Posted by "Michael Bouschen (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/jira/browse/JDO-157?page=comments#action_12331895 ]
Michael Bouschen commented on JDO-157:
--------------------------------------
Looks good!
> Implement new JDO 2 query tests cases concerning keywords.
> ----------------------------------------------------------
>
> Key: JDO-157
> URL: http://issues.apache.org/jira/browse/JDO-157
> Project: JDO
> Type: New Feature
> Components: tck20
> Reporter: Michael Watzek
> Assignee: Michael Watzek
> Attachments: JDO-157.patch, JDO-157.patch2
>
> We need 4 new test classes, one for each of the following assertions:
> - A14.4-6: Keywords must not be used as package names, class names, parameter names, or variable names in queries.
> - A14.4-7: Keywords are permitted as field names only if they are on the right side of the "." in field access expressions.
> - A14.6.13-1: The String version of Query represents all query elements using a single string. The string contains the following structure.
> - A14.6.13-2: Keywords, identified above in bold, are either all upper-case or all lower-case. Keywords cannot be mixed case.
> Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#Keywords.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira