[jira] Commented: (HARMONY-966) [classlib][beans] final fixes for ExpressionTest (2)

["Alexei Zakharov (JIRA)" <ji...@apache.org>]

    [ http://issues.apache.org/jira/browse/HARMONY-966?page=comments#action_12425574 ] 
            
Alexei Zakharov commented on HARMONY-966:
-----------------------------------------

Mikhail, 

About the failures on RI: I have created the separate JIRA (HARMONY-1061) for this. This was my mistake indeed.

Thanks,
Alexei

> [classlib][beans] final fixes for ExpressionTest (2)
> ----------------------------------------------------
>
>                 Key: HARMONY-966
>                 URL: http://issues.apache.org/jira/browse/HARMONY-966
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>         Environment: WinXP
>            Reporter: Alexei Zakharov
>         Assigned To: Mikhail Loenko
>         Attachments: ExpressionTest2.patch
>
>
> The following previously failed tests are affected by the attached patch:
> testGetValue_UnboundedNullMethodName
> This test checks if NoSuchMethodException is thrown. But both RI and our implementation throws NullPointerException in this case. So I have changed the test to check for NullPointerException.
> testGetValue_UnboundedOverloadedMethodsNull
> fails on RI also, tests the order of elements returned by Class.getMethods(), Removed.
> testGetValue_UnboundedOverloadedConstructorsNull
> invalid test, fails on RI. Removed.
> testGetValue_UnboundedClassForName
> Bug in test - tries to call Class.forName() for class loaded by another class loader. Corrected. 
> testConstructor_Value_SetValueCalledIn
> Tests implementation specific details of the third-party implementation. Fails on RI also. REmoved.
> This is the final fix for ExpressionsTest class from me. Now all tests from this class pass on the our implementation as well as on RI. We can finally remove this class from the exclude list.

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