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 Bouschen (JIRA)" <ji...@apache.org> on 2005/10/12 15:18:07 UTC
[jira] Commented: (JDO-157) Implement new JDO 2 query tests cases concerning keywords.
[ 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