DO NOT REPLY [Bug 7282] New: - (Smoketest) ErrorListenerTest throws NPE after building bad stylesheet

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=7282>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=7282

(Smoketest) ErrorListenerTest throws NPE after building bad stylesheet

           Summary: (Smoketest) ErrorListenerTest throws NPE after building
                    bad stylesheet
           Product: XalanJ2
           Version: CurrentCVS
          Platform: PC
        OS/Version: Windows NT/2K
            Status: NEW
          Severity: Major
          Priority: Other
         Component: org.apache.xalan.transformer
        AssignedTo: xalan-dev@xml.apache.org
        ReportedBy: curcuru@apache.org


TestID: ErrorListenerTest.java r1.5 testCase1(line:232), testCase2(line:319)
This test has been causing the smoketest to fail in GUMP runs since 
shortly before Xerces moved to 2.0.1 (note that Xalan CVS still has Xerces 
2.0.0, so if you just use CVS it may pass; to reproduce, put xercesImpl.jar 
2.0.1 in your java/bin directory).  Hence some behavior change in Xerces 
between 2.0.0 and 2.0.1 is causing a change in Xalan's error handling behavior.

It appears that after attempting to build a stylesheet with errors in it using a 
n ErrorListener that always continues (i.e. it returns even after a fatal 
error), if after the call to newTransformer() etc. returns you then attempt to 
access the transformer or continue using it, we throw an NPE.

Given that this is a clear error condition, the test may not be valid; we should 
decide if we want to fix Xalan somehow to perform more nicely under error 
conditions, or make it policy that after an error we may barf and update the 
test to match.