This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-io.git
The following commit(s) were added to refs/heads/master by this push: new 6f0d8ed8f Better local variable names 6f0d8ed8f is described below commit 6f0d8ed8f188fdeaa01c52027731f2146a0ee2f6 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Fri Mar 14 14:34:36 2025 -0400 Better local variable names --- .../apache/commons/io/input/XmlStreamReaderTest.java | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/test/java/org/apache/commons/io/input/XmlStreamReaderTest.java b/src/test/java/org/apache/commons/io/input/XmlStreamReaderTest.java index 2f2ba2413..49f8bbfa0 100644 --- a/src/test/java/org/apache/commons/io/input/XmlStreamReaderTest.java +++ b/src/test/java/org/apache/commons/io/input/XmlStreamReaderTest.java @@ -182,10 +182,10 @@ protected InputStream getXmlInputStream(final String bomType, final String xmlTy return new ByteArrayInputStream(baos.toByteArray()); } - private void parseCharset(final String hdr, final String enc, final IOFunction<InputStream, XmlStreamReader> factory) throws Exception { - try (InputStream stream = new ByteArrayInputStream(hdr.getBytes(StandardCharsets.UTF_8))) { + private void parseCharset(final String value, final String charsetName, final IOFunction<InputStream, XmlStreamReader> factory) throws Exception { + try (InputStream stream = new ByteArrayInputStream(value.getBytes(StandardCharsets.UTF_8))) { try (XmlStreamReader xml = factory.apply(stream)) { - assertEquals(enc.toUpperCase(Locale.ROOT), xml.getEncoding(), enc); + assertEquals(charsetName.toUpperCase(Locale.ROOT), xml.getEncoding(), charsetName); } } } @@ -461,15 +461,15 @@ public void testHttpValid(final String cT, final String bomEnc, final String str @ParameterizedTest(name = "{0}") @MethodSource(CharsetsTest.AVAIL_CHARSETS) - public void testIO_815(final String csName) throws Exception { - final MessageFormat fmt = new MessageFormat("<?xml version=\"1.0\" encoding=''{0}''?>\n<root>text</root>"); + public void testIO_815(final String charsetName) throws Exception { + final MessageFormat messageFormat = new MessageFormat("<?xml version=\"1.0\" encoding=''{0}''?>\n<root>text</root>"); final IOFunction<InputStream, XmlStreamReader> factoryCtor = XmlStreamReader::new; final IOFunction<InputStream, XmlStreamReader> factoryBuilder = stream -> XmlStreamReader.builder().setInputStream(stream).get(); - parseCharset(fmt.format(new Object[] { csName }), csName, factoryCtor); - parseCharset(fmt.format(new Object[] { csName }), csName, factoryBuilder); - for (final String alias : Charset.forName(csName).aliases()) { - parseCharset(fmt.format(new Object[] { alias }), alias, factoryCtor); - parseCharset(fmt.format(new Object[] { alias }), alias, factoryBuilder); + parseCharset(messageFormat.format(new Object[] { charsetName }), charsetName, factoryCtor); + parseCharset(messageFormat.format(new Object[] { charsetName }), charsetName, factoryBuilder); + for (final String alias : Charset.forName(charsetName).aliases()) { + parseCharset(messageFormat.format(new Object[] { alias }), alias, factoryCtor); + parseCharset(messageFormat.format(new Object[] { alias }), alias, factoryBuilder); } }