Author: ggregory
Date: Sun Sep 2 18:34:30 2012
New Revision: 1380024
URL: http://svn.apache.org/viewvc?rev=1380024&view=rev
Log:
[CODEC-155] DigestUtils.getDigest(String) should throw IllegalArgumentException
instead of RuntimeException. Use JUnit 4 expected annotation to test instead of
try/catch block.
Modified:
commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/digest/DigestUtilsTest.java
Modified:
commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/digest/DigestUtilsTest.java
URL:
http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/digest/DigestUtilsTest.java?rev=1380024&r1=1380023&r2=1380024&view=diff
==============================================================================
---
commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/digest/DigestUtilsTest.java
(original)
+++
commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/digest/DigestUtilsTest.java
Sun Sep 2 18:34:30 2012
@@ -55,14 +55,9 @@ public class DigestUtilsTest {
assertNotNull(new DigestUtils());
}
- @Test
+ @Test(expected=IllegalArgumentException.class)
public void testInternalNoSuchAlgorithmException() {
- try {
- DigestUtils.getDigest("Bogus Bogus");
- fail("A RuntimeException should have been thrown.");
- } catch (RuntimeException e) {
- // Expected exception.
- }
+ DigestUtils.getDigest("Bogus Bogus");
}
@Test
