This is not correct. In this test case there is no property named
"CUSTOMER" defined. The issue is that you can't call getList() with a
property name that does not exist. When the code is changed to throw
null, some upstream code in the getList() implementation will return
the parent DataObject rather than null, eventually resulting in a
ClassCastException.

I'm going to re-enable the test and change the property to something
like "nonExistantProperty" to make this clearer.

Brent

On 11/14/06, Kelvin Goodson (JIRA) <[email protected]> wrote:
   [ 
http://issues.apache.org/jira/browse/TUSCANY-885?page=comments#action_12449772 ]

Kelvin Goodson commented on TUSCANY-885:
----------------------------------------

The fix applied would seem to have exposed an issue with the TestCase, as there would 
seem to be a !isMany property referenced by the "CUSTOMER" property name.  
Attempting to get this single valued property as a list is causing the problem.  I have 
commented out the test case a luciano's request.

> Property DataObject.getProperty(String propertyName) should return null if 
the property cannot be found
> 
-------------------------------------------------------------------------------------------------------
>
>                 Key: TUSCANY-885
>                 URL: http://issues.apache.org/jira/browse/TUSCANY-885
>             Project: Tuscany
>          Issue Type: Bug
>          Components: Java SDO Implementation
>    Affects Versions: Java-Mx
>            Reporter: Fuhwei Lwo
>         Assigned To: Kelvin Goodson
>            Priority: Minor
>             Fix For: Java-Mx
>
>         Attachments: tuscany-885.patch
>
>
> Based on SDO 2.01 specification, DataObject.getProperty(String propertyName) 
should return null if the property cannot be found.  Unfortunately, it returns 
IlegalArgumentException now.

--
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



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to