Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Message-ID: <1230963.1168725387538.JavaMail.jira@brutus>
Date: Sat, 13 Jan 2007 13:56:27 -0800 (PST)
From: "Mark Hindess (JIRA)" <jira@apache.org>
To: commits@harmony.apache.org
Subject: [jira] Commented: (HARMONY-2628) [classlib][swing]
 JSplitPane.add(null) does not throw unspecified NPE
In-Reply-To: <31733949.1165921221081.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/HARMONY-2628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12464535 ] 

Mark Hindess commented on HARMONY-2628:
---------------------------------------

If I apply the test patch *without* the fix, it passes?


> [classlib][swing] JSplitPane.add(null) does not throw unspecified NPE
> ---------------------------------------------------------------------
>
>                 Key: HARMONY-2628
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2628
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Denis Kishenko
>         Attachments: Harmony-2628-Container.patch, Harmony-2628-ContainerTest.patch
>
>
> javax.swing.JSplitPane.add(null) does not throw unspecified NPE while RI does.
> Trere is no mention of any exception in the specification.
> Compatibility issue.
> Test for reproducing:
> import junit.framework.TestCase;
> import javax.swing.*;
> import java.awt.Component;
> public class Test extends TestCase {       
>     public void testcase1() {
>         JSplitPane s = new JSplitPane();
>         try {
>             s.add(null);
>             fail("NPE should be thrown");
>         } catch ( NullPointerException e) { 
>           //expected
>         }
>     }                
> }
> Output on Sun 1.5
> =================
> .
> Time: 0.328
> OK (1 test)
> Output on Harmony:
> ==================
> .F
> Time: 0.921
> There was 1 failure:
> 1) testcase1(Test)junit.framework.AssertionFailedError: NPE should be thrown
>         at Test.testcase1(Test.java:11)
>         at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira