This is an automated email from the ASF dual-hosted git repository. garydgregory pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/commons-xml.git
commit c477116bc0e4e368f2b9c9c38b65309d0da8e791 Author: Gary Gregory <[email protected]> AuthorDate: Fri Aug 28 13:38:01 2026 -0400 Renames in test assertion messages. --- src/test/java/org/apache/commons/xml/AttackTestSupport.java | 2 +- src/test/java/org/apache/commons/xml/SchemaLocationDomTest.java | 2 +- src/test/java/org/apache/commons/xml/SchemaLocationSaxTest.java | 2 +- src/test/java/org/apache/commons/xml/XIncludeTest.java | 8 ++++---- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/test/java/org/apache/commons/xml/AttackTestSupport.java b/src/test/java/org/apache/commons/xml/AttackTestSupport.java index 6104239..46c0722 100644 --- a/src/test/java/org/apache/commons/xml/AttackTestSupport.java +++ b/src/test/java/org/apache/commons/xml/AttackTestSupport.java @@ -279,7 +279,7 @@ private static void assertNoLeakOrThrows(final ThrowingSupplier<String> action, * @param description short label naming the JAXP surface under test. */ private static void assertNoLeakStrict(final ThrowingSupplier<String> action, final String description) { - final String output = assertDoesNotThrow(action, "Hardened " + description + " parse must not throw"); + final String output = assertDoesNotThrow(action, "Secured " + description + " parse must not throw"); assertFalse(output.contains(LEAKED_MARKER), "Securing did not block " + description + "; output contained marker '" + LEAKED_MARKER + "'.\nFull output:\n" + output); } diff --git a/src/test/java/org/apache/commons/xml/SchemaLocationDomTest.java b/src/test/java/org/apache/commons/xml/SchemaLocationDomTest.java index 904f72a..851432d 100644 --- a/src/test/java/org/apache/commons/xml/SchemaLocationDomTest.java +++ b/src/test/java/org/apache/commons/xml/SchemaLocationDomTest.java @@ -93,7 +93,7 @@ void hardenedDoesNotFetchExternalSchema() { try { final Document document = parse(factory); assertNotEquals(LEAKED_MARKER, document.getDocumentElement().getAttribute("leak"), - "Hardened parse must not inline the external schema's default attribute."); + "Secured parse must not inline the external schema's default attribute."); } catch (final Exception blocked) { // Acceptable: the empty schema was rejected at parse time, so nothing was fetched or inlined. } diff --git a/src/test/java/org/apache/commons/xml/SchemaLocationSaxTest.java b/src/test/java/org/apache/commons/xml/SchemaLocationSaxTest.java index f8a649b..1a682b7 100644 --- a/src/test/java/org/apache/commons/xml/SchemaLocationSaxTest.java +++ b/src/test/java/org/apache/commons/xml/SchemaLocationSaxTest.java @@ -109,7 +109,7 @@ void hardenedDoesNotFetchExternalSchema() throws Exception { } catch (final Exception blocked) { // Acceptable: the empty schema was rejected at parse time, so nothing was fetched or augmented. } - assertNull(handler.leak, "Hardened parse must not augment the external schema's default attribute onto the element."); + assertNull(handler.leak, "Secured parse must not augment the external schema's default attribute onto the element."); } @Test diff --git a/src/test/java/org/apache/commons/xml/XIncludeTest.java b/src/test/java/org/apache/commons/xml/XIncludeTest.java index 44ca989..6982848 100644 --- a/src/test/java/org/apache/commons/xml/XIncludeTest.java +++ b/src/test/java/org/apache/commons/xml/XIncludeTest.java @@ -193,7 +193,7 @@ void hardenedDomBlocksParseText() throws Exception { final Document doc = factory.newDocumentBuilder().parse(input); final String text = doc.getDocumentElement().getTextContent(); assertFalse(text.contains(LEAKED_MARKER), - "Hardened DOM parse=text must resolve the include to empty, not leak; got: " + text); + "Secured DOM parse=text must resolve the include to empty, not leak; got: " + text); } @Test @@ -207,7 +207,7 @@ void hardenedDomBlocksParseXml() throws Exception { assertThrows(SAXException.class, () -> { final DocumentBuilder builder = factory.newDocumentBuilder(); builder.parse(input); - }, "Hardened DOM parse=xml should throw"); + }, "Secured DOM parse=xml should throw"); } @Test @@ -268,7 +268,7 @@ void hardenedSaxBlocksParseText() throws Exception { assumeXIncludeAware(factory); final String captured = captureCharacters(factory.newSAXParser().getXMLReader(), input); assertFalse(captured.contains(LEAKED_MARKER), - "Hardened SAX parse=text must resolve the include to empty, not leak; got: " + captured); + "Secured SAX parse=text must resolve the include to empty, not leak; got: " + captured); } @Test @@ -282,7 +282,7 @@ void hardenedSaxBlocksParseXml() throws Exception { assertThrows(SAXException.class, () -> { final XMLReader reader = factory.newSAXParser().getXMLReader(); reader.parse(input); - }, "Hardened SAX parse=xml should throw"); + }, "Secured SAX parse=xml should throw"); } @Test
