--- old/test/javax/xml/jaxp/functional/javax/xml/transform/ptests/SAXTFactoryTest012.java 2014-12-31 11:40:11.924043007 -0800 +++ new/test/javax/xml/jaxp/functional/javax/xml/transform/ptests/SAXTFactoryTest012.java 2014-12-31 11:40:11.848042889 -0800 @@ -24,9 +24,6 @@ import java.io.FileInputStream; import java.io.IOException; -import java.nio.file.Files; -import java.nio.file.Path; -import java.nio.file.Paths; import javax.xml.transform.TransformerConfigurationException; import javax.xml.transform.TransformerFactory; import static javax.xml.transform.ptests.TransformerTestConst.CLASS_DIR; @@ -34,9 +31,8 @@ import static javax.xml.transform.ptests.TransformerTestConst.XML_DIR; import javax.xml.transform.sax.SAXSource; import javax.xml.transform.sax.SAXTransformerFactory; +import jaxp.library.JAXPFileBaseTest; import static jaxp.library.JAXPTestUtilities.compareWithGold; -import static jaxp.library.JAXPTestUtilities.failCleanup; -import static jaxp.library.JAXPTestUtilities.failUnexpected; import static org.testng.Assert.assertTrue; import org.testng.annotations.Test; import org.xml.sax.InputSource; @@ -49,45 +45,41 @@ * Test XMLFilter parse InputSource along with customized ContentHandler by * using SAX parser as it's reader. */ -public class SAXTFactoryTest012 { +public class SAXTFactoryTest012 extends JAXPFileBaseTest { /** * Unit test for contentHandler setter/getter. + * + * @throws SAXException If any parse errors occur. + * @throws IOException if the file exists but is a directory rather than + * a regular file, does not exist but cannot be created, or cannot + * be opened for any other reason. + * @throws TransformerConfigurationException If for some reason the + * TransformerHandler can not be created. */ @Test - public void testcase01() { + public void testcase01() throws SAXException, IOException, + TransformerConfigurationException { String outputFile = CLASS_DIR + "saxtf012.out"; String goldFile = GOLDEN_DIR + "saxtf012GF.out"; String xsltFile = XML_DIR + "cities.xsl"; String xmlFile = XML_DIR + "cities.xml"; - try { - // The transformer will use a SAX parser as it's reader. - XMLReader reader = XMLReaderFactory.createXMLReader(); - - InputSource is = new InputSource(new FileInputStream(xsltFile)); - SAXSource saxSource = new SAXSource(); - saxSource.setInputSource(is); - - SAXTransformerFactory saxTFactory = (SAXTransformerFactory)TransformerFactory.newInstance(); - XMLFilter filter = saxTFactory.newXMLFilter(saxSource); - - filter.setParent(reader); - filter.setContentHandler(new MyContentHandler(outputFile)); - - // Now, when you call transformer.parse, it will set itself as - // the content handler for the parser object (it's "parent"), and - // will then call the parse method on the parser. - filter.parse(new InputSource(xmlFile)); - assertTrue(compareWithGold(goldFile, outputFile)); - } catch (SAXException | IOException | TransformerConfigurationException ex) { - failUnexpected(ex); - } finally { - try { - Path outputPath = Paths.get(outputFile); - if(Files.exists(outputPath)) - Files.delete(outputPath); - } catch (IOException ex) { - failCleanup(ex, outputFile); - } - } + // The transformer will use a SAX parser as it's reader. + XMLReader reader = XMLReaderFactory.createXMLReader(); + + InputSource is = new InputSource(new FileInputStream(xsltFile)); + SAXSource saxSource = new SAXSource(); + saxSource.setInputSource(is); + + SAXTransformerFactory saxTFactory = (SAXTransformerFactory)TransformerFactory.newInstance(); + XMLFilter filter = saxTFactory.newXMLFilter(saxSource); + + filter.setParent(reader); + filter.setContentHandler(new MyContentHandler(outputFile)); + + // Now, when you call transformer.parse, it will set itself as + // the content handler for the parser object (it's "parent"), and + // will then call the parse method on the parser. + filter.parse(new InputSource(xmlFile)); + assertTrue(compareWithGold(goldFile, outputFile)); } }