Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312051 - some WDF test fail to detect an invalid query
Summary: some WDF test fail to detect an invalid query
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows Vista
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard: test
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-07 09:01 EDT by Adrian Goerler CLA
Modified: 2022-06-09 10:20 EDT (History)
1 user (show)

See Also:


Attachments
patch for this issue (7.20 KB, patch)
2010-05-07 09:03 EDT, Adrian Goerler CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Goerler CLA 2010-05-07 09:01:12 EDT
The test 

jpa/eclipselink.jpa.wdf.test
- TestConditionalExpressions
  - testBetweenHandling1
  - testBetweenHandling4
  - testBetweenHandling8
  - testBetweenHandling11
- TestGroupByOrderByHaving
  - testGroupBy1 
  - testGroupBy2

fail to detect an invalid query. This is intended and as the type checking has been loosened. The test have got to be adapted to assert that the query under test is valid.

Moreover, the test 
- TestBuiltInFunctionsExpressions
  - testConcatHandling3
  - testUpperHandling42
  - testLengthHandling50

are skipped currently but can be altered to assert that the query under test is valid.
Comment 1 Adrian Goerler CLA 2010-05-07 09:03:05 EDT
Created attachment 167463 [details]
patch for this issue

tested on MySQL -> OK
Comment 2 Adrian Goerler CLA 2010-05-10 09:18:23 EDT
Reviewed by James
Tested on MySQL -> OK
Checked in at rev 7188
Comment 3 Eclipse Webmaster CLA 2022-06-09 10:14:31 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink
Comment 4 Eclipse Webmaster CLA 2022-06-09 10:20:15 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink