This incompatibility is obviously caused by RI's assert usage, whether or not we need to be "assert compatible" with RI? assert on/off in Java is runtime decided, so that it's still possible that user application run JRE in assert enable mode. And if we do want to "assert compatible", we need to run our test suites in both mode.

IMHO, "assert compatibility" is not necessary, because it is not likely to cause user application fail, it seems crazy to expect assert error in the application logic. And I don't believe there are JCK test cases against assert, but I'm open to other's comments.

Artem Aliev (JIRA) wrote:
[luni]  java.util.prefs.AbstractPreferences.exportNode(OutputStream) and 
java.util.prefs.AbstractPreferences.exportSubtree(OutputStream) throws 
AssertionError instead of NPE in a  case when OutputStream is null
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

                 Key: HARMONY-2420
                 URL: http://issues.apache.org/jira/browse/HARMONY-2420
             Project: Harmony
          Issue Type: Bug
          Components: Classlib
            Reporter: Artem Aliev
            Priority: Minor


exportNode(OutputStream) and exportSubtree(OutputStream) throws
AssertionError not NullPointerException in a case when OutputStream is null.

Test to reproduce:
import junit.framework.TestCase;

import java.io.IOException;
import java.util.prefs.*;

class AbstractPreferencesImpl extends AbstractPreferences{

        protected AbstractPreferencesImpl(AbstractPreferences arg0, String
arg1) {
                super(arg0, arg1);
        }

        protected AbstractPreferences childSpi(String arg0) {
                return null;
        }

        protected String[] childrenNamesSpi() throws BackingStoreException {
                return null;
        }

        protected void flushSpi() throws BackingStoreException {
        }

        protected String getSpi(String arg0) {
                return null;
        }

        protected String[] keysSpi() throws BackingStoreException {
                return null;
        }

        protected void putSpi(String arg0, String arg1) {
        }

        protected void removeNodeSpi() throws BackingStoreException {
        }

        protected void removeSpi(String arg0) {
        }

        protected void syncSpi() throws BackingStoreException {
        }
}

public class Test extends TestCase {

        public void testcase() {
                AbstractPreferences ap = new AbstractPreferencesImpl(null, "");
                boolean res1 = false, res2 = false;
                try {
                        ap.exportNode(null);
                } catch (IOException e) {
                        e.printStackTrace();
                } catch (BackingStoreException e) {
                        e.printStackTrace();
                }catch (NullPointerException e) {
                        e.printStackTrace();
                        System.out.println("Got Expected
NullPointerException");
                } catch (AssertionError ae) {
                        ae.printStackTrace();
                        res1 = true;
                        System.out.println("Got Unexpected AssertionError");
} try {
                        ap.exportSubtree(null);
                } catch (IOException e) {
                        e.printStackTrace();
                } catch (BackingStoreException e) {
                        e.printStackTrace();
                }catch (NullPointerException e) {
                        e.printStackTrace();
                        System.out.println("Got Expected
NullPointerException");
                } catch (AssertionError ae) {
                        ae.printStackTrace();
                        res2 = true;
                        System.out.println("Got Unexpected AssertionError");
} if (res1 || res2) {
                        fail();
                }
    }
}

Output on Harmony:
Apache Harmony Launcher : (c) Copyright 1991, 2006 The Apache Software
Foundation or its licensors, as applicable.
java version "1.5.0" pre-alpha : not complete or compatible
svn = rsvn: '.' is not a working copy, (Sep 20 2006), Windows/ia32/msvc 1310,
release build
http://incubator.apache.org/harmony

.java.lang.AssertionError: Stream is null
        at
java.util.prefs.AbstractPreferences.exportNode(AbstractPreferences.java:373)
        at Test.testcase(Test.java:50)
        at java.lang.reflect.VMReflection.invokeMethod(Native Method)
        at java.lang.reflect.Method.invoke(Unknown Source)
        at junit.framework.TestCase.runTest(TestCase.java:154)
        at junit.framework.TestCase.runBare(TestCase.java:127)
        at junit.framework.TestResult$1.protect(TestResult.java:106)
        at junit.framework.TestResult.runProtected(TestResult.java:124)
        at junit.framework.TestResult.run(TestResult.java:109)
        at junit.framework.TestCase.run(TestCase.java:118)
        at junit.framework.TestSuite.runTest(TestSuite.java:208)
        at junit.framework.TestSuite.run(TestSuite.java:203)
        at junit.textui.TestRunner.doRun(TestRunner.java:115)
        at junit.textui.TestRunner.start(TestRunner.java:172)
        at junit.textui.TestRunner.main(TestRunner.java:138)
Got Unexpected AssertionError
java.lang.AssertionError: Stream is null
        at
java.util.prefs.AbstractPreferences.exportSubtree(AbstractPreferences.java:388)
        at Test.testcase(Test.java:64)
        at java.lang.reflect.VMReflection.invokeMethod(Native Method)
        at java.lang.reflect.Method.invoke(Unknown Source)
        at junit.framework.TestCase.runTest(TestCase.java:154)
        at junit.framework.TestCase.runBare(TestCase.java:127)
        at junit.framework.TestResult$1.protect(TestResult.java:106)
        at junit.framework.TestResult.runProtected(TestResult.java:124)
        at junit.framework.TestResult.run(TestResult.java:109)
        at junit.framework.TestCase.run(TestCase.java:118)
        at junit.framework.TestSuite.runTest(TestSuite.java:208)
        at junit.framework.TestSuite.run(TestSuite.java:203)
        at junit.textui.TestRunner.doRun(TestRunner.java:115)
        at junit.textui.TestRunner.start(TestRunner.java:172)
        at junit.textui.TestRunner.main(TestRunner.java:138)
Got Unexpected AssertionError
F
Time: 0.125
There was 1 failure:
1) testcase(Test)junit.framework.AssertionFailedError
        at Test.testcase(Test.java:78)
        at java.lang.reflect.VMReflection.invokeMethod(Native Method)

FAILURES!!!
Tests run: 1,  Failures: 1,  Errors: 0

res = 1


Output on RI:
java version "1.5.0_06"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_06-b05)
Java HotSpot(TM) Client VM (build 1.5.0_06-b05, mixed mode)
.java.lang.NullPointerException
        at java.util.prefs.XmlSupport.putPreferencesInXml(XmlSupport.java:140)
        at java.util.prefs.XmlSupport.export(XmlSupport.java:104)
        at
java.util.prefs.AbstractPreferences.exportNode(AbstractPreferences.java:1575)
        at Test.testcase(Test.java:50)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:585)
        at junit.framework.TestCase.runTest(TestCase.java:154)
        at junit.framework.TestCase.runBare(TestCase.java:127)
        at junit.framework.TestResult$1.protect(TestResult.java:106)
        at junit.framework.TestResult.runProtected(TestResult.java:124)
        at junit.framework.TestResult.run(TestResult.java:109)
        at junit.framework.TestCase.run(TestCase.java:118)
        at junit.framework.TestSuite.runTest(TestSuite.java:208)
        at junit.framework.TestSuite.run(TestSuite.java:203)
        at junit.textui.TestRunner.doRun(TestRunner.java:116)
        at junit.textui.TestRunner.start(TestRunner.java:172)
        at junit.textui.TestRunner.main(TestRunner.java:138)
Got Expected NullPointerException
java.lang.NullPointerException
        at java.util.prefs.XmlSupport.putPreferencesInXml(XmlSupport.java:140)
        at java.util.prefs.XmlSupport.export(XmlSupport.java:104)
        at
java.util.prefs.AbstractPreferences.exportSubtree(AbstractPreferences.java:1591)
        at Test.testcase(Test.java:64)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:585)
        at junit.framework.TestCase.runTest(TestCase.java:154)
        at junit.framework.TestCase.runBare(TestCase.java:127)
        at junit.framework.TestResult$1.protect(TestResult.java:106)
        at junit.framework.TestResult.runProtected(TestResult.java:124)
        at junit.framework.TestResult.run(TestResult.java:109)
        at junit.framework.TestCase.run(TestCase.java:118)
        at junit.framework.TestSuite.runTest(TestSuite.java:208)
        at junit.framework.TestSuite.run(TestSuite.java:203)
        at junit.textui.TestRunner.doRun(TestRunner.java:116)
        at junit.textui.TestRunner.start(TestRunner.java:172)
        at junit.textui.TestRunner.main(TestRunner.java:138)
Got Expected NullPointerException

Time: 0.078

OK (1 test)

res = 0




--
Paulex Yang
China Software Development Lab
IBM


Reply via email to