garydgregory commented on a change in pull request #113:
URL: https://github.com/apache/commons-codec/pull/113#discussion_r811945196
##########
File path:
src/test/java/org/apache/commons/codec/binary/Base16InputStreamTest.java
##########
@@ -327,33 +323,21 @@ public void testReadOutOfBounds() throws IOException {
final ByteArrayInputStream bin = new ByteArrayInputStream(decoded);
try (final Base16InputStream in = new Base16InputStream(bin, true)) {
- try {
- in.read(buf, -1, 0);
- fail("Expected Base16InputStream.read(buf, -1, 0) to throw
IndexOutOfBoundsException");
- } catch (final IndexOutOfBoundsException e) {
- // Expected
- }
+ assertThrows(IndexOutOfBoundsException.class, () -> {
Review comment:
@itamarc
Same comment as for the other extra {} in other files...
TY!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
