[#JDO-320] Add negative query test for projecting a map and collection field to tck20

JDO

Add negative query test for projecting a map and collection field to tck20

Created: 26/Feb/06 07:38 AM Updated: 15/Mar/06 06:33 AM

Return to search

Component/s:

tck2

Affects Version/s:

JDO 2 rc1

Fix Version/s:

JDO 2 final

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	JDO-320.patch	2006-03-03 06:23 AM	Michael Bouschen	2 kB

Resolution Date:

15/Mar/06 06:33 AM

Description

« Hide

The latest version of the JDO spec disallows JDOQL queries projecting a collection or map field, e.g.
SELECT p.phoneNumbers FROM org.apache.jdo.tck.pc.company.Person p
SELECT d.employees FROM org.apache.jdo.tck.pc.company.Department d
We should add a test method running the above queries which should result in a JDOException.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Michael Bouschen added a comment - 03/Mar/06 06:23 AM

Attached you find a patch ~~JDO-320~~.patch for review. It adds the two negative test queries to the ResultExpressions class.

Please note, that the current JPOX version currently accepts JDOQL queries returning a collection field, so the negative test fails.

[ Permlink | « Hide ]

Craig Russell added a comment - 03/Mar/06 01:20 PM

Looks good.

[ Permlink | « Hide ]

Michael Bouschen added a comment - 05/Mar/06 04:57 AM

Checked in the patch which adds two negative test queries to class org.apache.jdo.tck.query.result.ResultExpressions (see revision 383191).

Assigning the issue to Erik, becaue the current JPOX version allows projection of collection and map fields, so the negative tests fail.

[ Permlink | « Hide ]

Erik Bengtson added a comment - 08/Mar/06 05:33 PM

should be fixed in JPOX CVS now

[ Permlink | « Hide ]

Craig Russell added a comment - 15/Mar/06 06:33 AM

Fixed with the latest JPOX build.