You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Alexei Zakharov (JIRA)" <ji...@apache.org> on 2006/07/19 15:50:14 UTC

[jira] Updated: (HARMONY-924) [classlib][beans] some tests from ExpressionTest class fail

     [ http://issues.apache.org/jira/browse/HARMONY-924?page=all ]

Alexei Zakharov updated HARMONY-924:
------------------------------------

    Attachment: ExpressionTest.patch

This should be applied from the modules/beans/src level.

> [classlib][beans] some tests from ExpressionTest class fail
> -----------------------------------------------------------
>
>                 Key: HARMONY-924
>                 URL: http://issues.apache.org/jira/browse/HARMONY-924
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>         Environment: winXP
>            Reporter: Alexei Zakharov
>         Attachments: ExpressionTest.patch
>
>
> Currently 23 out of 56 tests from org.apache.harmony.beans.tests.java.beans.ExpressionTest class fail due various reasons.  Attached patch corrects following invalid tests:
> testConstructor_Normal took
> testConstructor_NullTarget took
> testConstructor_ArrayTarget took
> testConstructor_NullMethodName
> testConstructor_NewMethodName
> testConstructor_EmptyMethodName
> testConstructor_NullArguments took
> testConstructor_NullArgument took
> testConstructor_Value_Normal took
> testConstructor_Value_NullTarget took
> testConstructor_Value_ArrayTarget took
> testConstructor_Value_NullMethodName took 
> testConstructor_Value_NewMethodName took
> testConstructor_Value_EmptyMethodName took
> testConstructor_Value_NullArguments took
> testConstructor_Value_NullArgument took
> testConstructor_Value_NullValue took
> These tests fail on RI also.
> Change log:
> 1. All tests above checked for invalid toString() patterns. Corrected. Now it checks for patterns returned by RI. 
> 2. Some tests constructed Expression without return value specified and then called toString(). Both our implementation and RI execute the expression in this case to obtain the return value. But it seems these tests didn't expect Statement.execute() method to be called and didn't pass valid parameters to Statement constructs, just stubs. This situation has been correct.
> 3. For some tests calling toString is an invalid action at all. I have just removed this assertion for these tests.
> As a result all tests mentioned above pass on our implementation as well as on RI.

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