Return-Path: Delivered-To: apmail-incubator-harmony-commits-archive@www.apache.org Received: (qmail 13285 invoked from network); 1 Aug 2006 16:12:47 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 1 Aug 2006 16:12:47 -0000 Received: (qmail 40638 invoked by uid 500); 1 Aug 2006 16:12:47 -0000 Delivered-To: apmail-incubator-harmony-commits-archive@incubator.apache.org Received: (qmail 40599 invoked by uid 500); 1 Aug 2006 16:12:47 -0000 Mailing-List: contact harmony-commits-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: harmony-dev@incubator.apache.org Delivered-To: mailing list harmony-commits@incubator.apache.org Received: (qmail 40588 invoked by uid 99); 1 Aug 2006 16:12:47 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Aug 2006 09:12:47 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [209.237.227.198] (HELO brutus.apache.org) (209.237.227.198) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Aug 2006 09:12:46 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 4C3DF410011 for ; Tue, 1 Aug 2006 16:10:15 +0000 (GMT) Message-ID: <30600685.1154448615309.JavaMail.jira@brutus> Date: Tue, 1 Aug 2006 09:10:15 -0700 (PDT) From: "Alexei Zakharov (JIRA)" To: harmony-commits@incubator.apache.org Subject: [jira] Updated: (HARMONY-975) [classlib][beans] fixes for DefaultPersistenceDelegateTest failures (1) In-Reply-To: <29032437.1153844774025.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N [ http://issues.apache.org/jira/browse/HARMONY-975?page=all ] Alexei Zakharov updated HARMONY-975: ------------------------------------ Attachment: DefailtPersistenceDelegateTest2.patch Stepan, please see the new patch attached. It enables all tests mentioned above on our implementation (no corrections in tests). However, testInstantiate_EmptyProperty still fails on the RI. It seems the behaviour of our Introspector differs from RI's Introspector. I think I will create a separate JIRA for that. Thanks. > [classlib][beans] fixes for DefaultPersistenceDelegateTest failures (1) > ----------------------------------------------------------------------- > > Key: HARMONY-975 > URL: http://issues.apache.org/jira/browse/HARMONY-975 > Project: Harmony > Issue Type: Improvement > Components: Classlib > Environment: winXP > Reporter: Alexei Zakharov > Attachments: DefailtPersistenceDelegateTest.patch, DefailtPersistenceDelegateTest2.patch > > > Currently 19 tests from org.apache.harmony.beans.tests.java.beans.DefaultPersistenceDelegateTest class fail due various reasons. The attached patch corrects some of these failures. Please see the list of affected tests below. > testInstantiate_NullProperty > testInstantiate_EmptyProperty > testInstantiate_NonExistingGetterNullEncoder > Invalid tests indeed. Fail if NullPointerException isn't thrown. Neither RI nor our implementation throws such exception here. Corrected. > testInstantiate_NonExistingGetter > testInstantiate_NoGetter > Are also invalid tests. They test the behavior that is specific to some third-party implementation of beans (not RI). I corrected the sophisticated but invalid exception handling I've found in these tests. Now it reflects the RI behavior. Please note that using of singleton class CallVerificationStack is thread unsafe here. I have replaced CallVerificationStack with locally defined variable. -- 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