You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Pavel Dolgov (JIRA)" <ji...@apache.org> on 2006/12/04 17:41:21 UTC

[jira] Created: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.dispatchEvent(null) does not throw NPE

[classlib][awt] java.awt.DefaultKeyboardFocusManager.dispatchEvent(null) does not throw NPE
-------------------------------------------------------------------------------------------

                 Key: HARMONY-2446
                 URL: http://issues.apache.org/jira/browse/HARMONY-2446
             Project: Harmony
          Issue Type: Bug
          Components: Classlib
            Reporter: Pavel Dolgov
            Priority: Minor


Test case:
----------------
import java.awt.DefaultKeyboardFocusManager;
import java.awt.KeyboardFocusManager;

import junit.framework.TestCase;

public class TheTest extends TestCase {

    public static void main(String args[]) {
        junit.textui.TestRunner.run(TheTest.class);
    }

    public void testcase1() {
        DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
        try {
            dkm.setDefaultFocusTraversalKeys(
                    KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
            fail("IAE expected");
        } catch (IllegalArgumentException e) {
        } catch (Exception e) {
            fail(e.getMessage());
        }
    }
}

RI output:
--------------
.
Time: 0,701

OK (1 test)

Harmony output:
--------------
.F
Time: 0.491
There was 1 failure:
1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
	at TheTest.testcase1(TheTest.java:17)
	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
	at TheTest.main(TheTest.java:9)

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: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Alexei Zakharov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-2446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexei Zakharov updated HARMONY-2446:
-------------------------------------

    Patch Info: [Patch Available]

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>            Priority: Minor
>         Attachments: KeyboardFocusManager.patch, KeyboardFocusManagerRTest.patch
>
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> 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

        

[jira] Updated: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Pavel Dolgov (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-2446?page=all ]

Pavel Dolgov updated HARMONY-2446:
----------------------------------

    Summary: [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null  (was: [classlib][awt] java.awt.DefaultKeyboardFocusManager.dispatchEvent(null) does not throw NPE)

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>            Priority: Minor
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> 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: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Andrey Pavlenko (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-2446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12467129 ] 

Andrey Pavlenko commented on HARMONY-2446:
------------------------------------------

Alexei, this issues is already fixed in HARMONY-2493. You can close this issue. 

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>         Assigned To: Alexei Zakharov
>            Priority: Minor
>         Attachments: KeyboardFocusManager.patch, KeyboardFocusManagerRTest.patch
>
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Alexei Zakharov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-2446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexei Zakharov reassigned HARMONY-2446:
----------------------------------------

    Assignee: Alexei Zakharov

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>         Assigned To: Alexei Zakharov
>            Priority: Minor
>         Attachments: KeyboardFocusManager.patch, KeyboardFocusManagerRTest.patch
>
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Alexei Zakharov (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-2446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12467100 ] 

Alexei Zakharov commented on HARMONY-2446:
------------------------------------------

I am unable to reproduce the failure using the attached test neither on Windows nor on Linux.

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>         Assigned To: Alexei Zakharov
>            Priority: Minor
>         Attachments: KeyboardFocusManager.patch, KeyboardFocusManagerRTest.patch
>
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Alexei Zakharov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-2446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexei Zakharov closed HARMONY-2446.
------------------------------------

    Resolution: Fixed

Fixed in HARMONY-2493

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>         Assigned To: Alexei Zakharov
>            Priority: Minor
>         Attachments: KeyboardFocusManager.patch, KeyboardFocusManagerRTest.patch
>
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-2446) [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null

Posted by "Andrey Pavlenko (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-2446?page=all ]

Andrey Pavlenko updated HARMONY-2446:
-------------------------------------

    Attachment: KeyboardFocusManager.patch
                KeyboardFocusManagerRTest.patch

The patch and the regression test are attached.

> [classlib][awt] java.awt.DefaultKeyboardFocusManager.setDefaultFocusTraversalKeys() does not throw IllegalArgumentException if second argument is null
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2446
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2446
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>            Priority: Minor
>         Attachments: KeyboardFocusManager.patch, KeyboardFocusManagerRTest.patch
>
>
> Test case:
> ----------------
> import java.awt.DefaultKeyboardFocusManager;
> import java.awt.KeyboardFocusManager;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         DefaultKeyboardFocusManager dkm = new DefaultKeyboardFocusManager();
>         try {
>             dkm.setDefaultFocusTraversalKeys(
>                     KeyboardFocusManager.BACKWARD_TRAVERSAL_KEYS, null);
>             fail("IAE expected");
>         } catch (IllegalArgumentException e) {
>         } catch (Exception e) {
>             fail(e.getMessage());
>         }
>     }
> }
> RI output:
> --------------
> .
> Time: 0,701
> OK (1 test)
> Harmony output:
> --------------
> .F
> Time: 0.491
> There was 1 failure:
> 1) testcase1(TheTest)junit.framework.AssertionFailedError: IAE expected
> 	at TheTest.testcase1(TheTest.java:17)
> 	at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> 	at TheTest.main(TheTest.java:9)
> 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: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira