[#JDO-158] Implement new JDO 2 query tests cases concerning new operators.

JDO

Implement new JDO 2 query tests cases concerning new operators.

Created: 04/Oct/05 08:04 PM Updated: 13/Oct/05 05:50 PM

Return to search

Component/s:

tck2

Affects Version/s:

None

Fix Version/s:

None

Time Tracking:

Not Specified

File Attachments:

File Name	Date Attached	Attached By	Size
JDO-158.patch	2005-10-06 07:15 PM	Michael Watzek	8 kB
JDO-158.patch2	2005-10-13 01:30 AM	Michael Watzek	10 kB

Resolution Date:

13/Oct/05 05:50 PM

Description

« Hide

We need 2 new test classes, one for each of the following assertions:

- A14.6.2-40: modulo operator.
- A14.6.2-41: instanceof operator.

Details can be found on Wiki page http://wiki.apache.org/jdo/QueryTests#NewOperators.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Michael Watzek added a comment - 06/Oct/05 07:15 PM

This patch implements the assertions above. It depends on patch for ~~JDO-157~~. For this reason, that patch must be checked in first.

[ Permlink | « Hide ]

Michael Bouschen added a comment - 12/Oct/05 10:56 PM

- A14.6.2-40: modulo operator.
The current implementation uses a QueryElementHolder to run a modulo query as API query and single string Query. This is godd! How about adding another test method that runs modulo queries for all integral data types? Please have a look at the other test classes in the operators package, e.g. class Multiplication.

- A14.6.2-41: instanceof operator.
How about adding a second test query that import class PartTimeEmployee and uses the unqualified class name in the instanceof operator?

[ Permlink | « Hide ]

Michael Watzek added a comment - 13/Oct/05 01:30 AM

The second patch contains all the comments.

[ Permlink | « Hide ]

Michael Bouschen added a comment - 13/Oct/05 05:43 PM

Looks good, ship it!

[ Permlink | « Hide ]

Michael Watzek added a comment - 13/Oct/05 05:50 PM

The second patch has been checked in.