You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2023/02/09 16:17:53 UTC
[commons-beanutils] branch master updated (a552180c -> 3526b161)
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/commons-beanutils.git
from a552180c Remove extra whitespace
new b6bdccad Remove extra whitespace
new a4433cd2 Rename test class to match class under test
new eed83986 Rename test class to match class under test
new 2cd4830b Remove extra whitespace
new e833b2c5 Generics
new 3526b161 Generics
The 6 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../apache/commons/beanutils2/ConvertUtils.java | 6 ++--
.../commons/beanutils2/ConvertUtilsBean.java | 35 +++++++++++-----------
.../commons/beanutils2/ConvertUtilsBean2.java | 2 +-
.../apache/commons/beanutils2/BeanMapTestCase.java | 6 ++--
...s2TestCase.java => BeanUtilsBean2TestCase.java} | 10 ++-----
...ilsTestCase.java => BeanUtilsBeanTestCase.java} | 8 ++---
.../commons/beanutils2/BeanUtilsBenchCase.java | 17 -----------
.../commons/beanutils2/BeanificationTestCase.java | 4 +--
.../commons/beanutils2/PropertyUtilsTestCase.java | 2 +-
.../locale/LocaleBeanificationTestCase.java | 6 ++--
.../memoryleaktests/MemoryLeakTestCase.java | 11 ++++---
11 files changed, 41 insertions(+), 66 deletions(-)
rename src/test/java/org/apache/commons/beanutils2/{BeanUtils2TestCase.java => BeanUtilsBean2TestCase.java} (97%)
rename src/test/java/org/apache/commons/beanutils2/{BeanUtilsTestCase.java => BeanUtilsBeanTestCase.java} (99%)
[commons-beanutils] 02/06: Rename test class to match class under test
Posted by gg...@apache.org.
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-beanutils.git
commit a4433cd222417b530c31d5956bb40510cd5dc275
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Feb 9 10:04:19 2023 -0500
Rename test class to match class under test
---
src/test/java/org/apache/commons/beanutils2/BeanMapTestCase.java | 6 +++---
src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java | 2 +-
.../{BeanUtilsTestCase.java => BeanUtilsBeanTestCase.java} | 6 +++---
.../java/org/apache/commons/beanutils2/BeanificationTestCase.java | 4 ++--
.../java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java | 2 +-
.../commons/beanutils2/locale/LocaleBeanificationTestCase.java | 6 +++---
6 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanMapTestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanMapTestCase.java
index e7b0116f..24599564 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanMapTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanMapTestCase.java
@@ -389,7 +389,7 @@ public class BeanMapTestCase extends AbstractTestMap {
*/
public void testExceptionThrowFromClone() {
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("testExceptionThrowFromClone() skipped on pre 1.4 JVM");
return;
}
@@ -433,7 +433,7 @@ public class BeanMapTestCase extends AbstractTestMap {
*/
public void testExceptionThrowFromClear() {
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("testExceptionThrowFromClear() skipped on pre 1.4 JVM");
return;
}
@@ -460,7 +460,7 @@ public class BeanMapTestCase extends AbstractTestMap {
*/
public void testExceptionThrowFromPut() {
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("testExceptionThrowFromPut() skipped on pre 1.4 JVM");
return;
}
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java
index be06c6c7..7680f046 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java
@@ -23,7 +23,7 @@ import junit.framework.TestSuite;
* Test Case for the {@link BeanUtilsBean2}.
*
*/
-public class BeanUtils2TestCase extends BeanUtilsTestCase {
+public class BeanUtils2TestCase extends BeanUtilsBeanTestCase {
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanUtilsBeanTestCase.java
similarity index 99%
rename from src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java
rename to src/test/java/org/apache/commons/beanutils2/BeanUtilsBeanTestCase.java
index e1adeed8..75dbed8f 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanUtilsBeanTestCase.java
@@ -58,7 +58,7 @@ import junit.framework.TestSuite;
* </ul>
*
*/
-public class BeanUtilsTestCase extends TestCase {
+public class BeanUtilsBeanTestCase extends TestCase {
/**
* The test bean for each test.
@@ -105,7 +105,7 @@ public class BeanUtilsTestCase extends TestCase {
*
* @param name Name of the test case
*/
- public BeanUtilsTestCase(final String name) {
+ public BeanUtilsBeanTestCase(final String name) {
super(name);
}
@@ -145,7 +145,7 @@ public class BeanUtilsTestCase extends TestCase {
* Creates the tests included in this test suite.
*/
public static Test suite() {
- return new TestSuite(BeanUtilsTestCase.class);
+ return new TestSuite(BeanUtilsBeanTestCase.class);
}
/**
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java
index b2508322..2a19fb0b 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java
@@ -108,7 +108,7 @@ public class BeanificationTestCase extends TestCase {
public void testMemoryLeak2() throws Exception {
// tests when the map used by beanutils has the right behavior
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("WARNING: CANNOT TEST MEMORY LEAK ON PRE1.4 JVM");
return;
}
@@ -157,7 +157,7 @@ public class BeanificationTestCase extends TestCase {
/** Tests whether classloaders and beans are released from memory */
public void testMemoryLeak() throws Exception {
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("WARNING: CANNOT TEST MEMORY LEAK ON PRE1.4 JVM");
return;
}
diff --git a/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java b/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java
index 9cbe08fc..ddc752ee 100644
--- a/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java
@@ -4324,7 +4324,7 @@ public class PropertyUtilsTestCase extends TestCase {
* See BEANUTILS-266 for changes and reason for test
*/
public void testExceptionFromInvoke() throws Exception {
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
return;
}
try {
diff --git a/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java b/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java
index 91713cb3..6e73db92 100644
--- a/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java
@@ -24,7 +24,7 @@ import java.util.Map;
import java.util.WeakHashMap;
import org.apache.commons.beanutils2.BeanUtilsBean;
-import org.apache.commons.beanutils2.BeanUtilsTestCase;
+import org.apache.commons.beanutils2.BeanUtilsBeanTestCase;
import org.apache.commons.beanutils2.ContextClassLoaderLocal;
import org.apache.commons.beanutils2.ConversionException;
import org.apache.commons.beanutils2.ConvertUtils;
@@ -114,7 +114,7 @@ public class LocaleBeanificationTestCase extends TestCase {
public void testMemoryLeak2() {
// tests when the map used by beanutils has the right behavior
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("WARNING: CANNOT TEST MEMORY LEAK ON PRE1.4 JVM");
return;
}
@@ -161,7 +161,7 @@ public class LocaleBeanificationTestCase extends TestCase {
/** Tests whether classloaders and beans are released from memory */
public void testMemoryLeak() throws Exception {
- if (BeanUtilsTestCase.isPre14JVM()) {
+ if (BeanUtilsBeanTestCase.isPre14JVM()) {
System.out.println("WARNING: CANNOT TEST MEMORY LEAK ON PRE1.4 JVM");
return;
}
[commons-beanutils] 06/06: Generics
Posted by gg...@apache.org.
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-beanutils.git
commit 3526b1613618c1d1c2d16559613a27d99ee4b95c
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Feb 9 11:17:47 2023 -0500
Generics
---
.../commons/beanutils2/ConvertUtilsBean.java | 35 +++++++++++-----------
.../commons/beanutils2/ConvertUtilsBean2.java | 2 +-
.../memoryleaktests/MemoryLeakTestCase.java | 11 ++++---
3 files changed, 23 insertions(+), 25 deletions(-)
diff --git a/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean.java b/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean.java
index 385db594..04cb7ee1 100644
--- a/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean.java
+++ b/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean.java
@@ -234,9 +234,9 @@ public class ConvertUtilsBean {
/**
* Convert the specified value to an object of the specified class (if
- * possible). Otherwise, return a String representation of the value.
+ * possible). Otherwise, return a {@link String} representation of the value.
*
- * @param <T> The desired return type
+ * @param <T> The <em>desired</em> return type
* @param value Value to be converted (may be null)
* @param clazz Java class to be converted to (must not be null)
* @return The converted value
@@ -274,20 +274,19 @@ public class ConvertUtilsBean {
*
* @throws ConversionException if thrown by an underlying Converter
*/
- public Object convert(final String[] values, final Class<?> clazz) {
-
- Class<?> type = clazz;
- if (clazz.isArray()) {
- type = clazz.getComponentType();
- }
+ public <T> Object convert(final String[] values, final Class<T> clazz) {
+ final Class<?> type = clazz.isArray() ? clazz.getComponentType() : clazz;
if (LOG.isDebugEnabled()) {
- LOG.debug("Convert String[" + values.length + "] to class '" +
- type.getName() + "[]'");
+ LOG.debug("Convert String[" + values.length + "] to class '" + type.getName() + "[]'");
}
Converter converter = lookup(type);
if (converter == null) {
converter = lookup(String.class);
}
+ return convert(values, type, converter);
+ }
+
+ private <T> Object convert(final String[] values, final Class<T> type, final Converter<T> converter) {
if (LOG.isTraceEnabled()) {
LOG.trace(" Using converter " + converter);
}
@@ -296,7 +295,6 @@ public class ConvertUtilsBean {
Array.set(array, i, converter.convert(type, values[i]));
}
return array;
-
}
/**
@@ -310,11 +308,12 @@ public class ConvertUtilsBean {
*
* @throws ConversionException if thrown by an underlying Converter
*/
- public Object convert(final Object value, final Class<?> targetType) {
- final Class<?> sourceType = value == null ? null : value.getClass();
+ public <T> Object convert(final Object value, final Class<T> targetType) {
+ final boolean nullValue = value == null;
+ final Class<?> sourceType = nullValue ? null : value.getClass();
if (LOG.isDebugEnabled()) {
- if (value == null) {
+ if (nullValue) {
LOG.debug("Convert null value to type '" + targetType.getName() + "'");
} else {
LOG.debug("Convert type '" + sourceType.getName() + "' value '" + value + "' to type '"
@@ -323,7 +322,7 @@ public class ConvertUtilsBean {
}
Object converted = value;
- Converter converter = lookup(sourceType, targetType);
+ Converter<T> converter = lookup(sourceType, targetType);
if (converter != null) {
if (LOG.isTraceEnabled()) {
LOG.trace(" Using converter " + converter);
@@ -335,12 +334,12 @@ public class ConvertUtilsBean {
// NOTE: For backwards compatibility, if the Converter
// doesn't handle conversion-->String then
// use the registered String Converter
- converter = lookup(String.class);
- if (converter != null) {
+ Converter<String> strConverter = lookup(String.class);
+ if (strConverter != null) {
if (LOG.isTraceEnabled()) {
LOG.trace(" Using converter " + converter);
}
- converted = converter.convert(String.class, converted);
+ converted = strConverter.convert(String.class, converted);
}
// If the object still isn't a String, use toString() method
diff --git a/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean2.java b/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean2.java
index 20526610..e268d49c 100644
--- a/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean2.java
+++ b/src/main/java/org/apache/commons/beanutils2/ConvertUtilsBean2.java
@@ -70,7 +70,7 @@ public class ConvertUtilsBean2 extends ConvertUtilsBean {
* @see ConvertUtilsBean#convert(String[], Class)
*/
@Override
- public Object convert(final String[] value, final Class<?> clazz) {
+ public <T> Object convert(final String[] value, final Class<T> clazz) {
return convert((Object) value, clazz);
}
diff --git a/src/test/java/org/apache/commons/beanutils2/memoryleaktests/MemoryLeakTestCase.java b/src/test/java/org/apache/commons/beanutils2/memoryleaktests/MemoryLeakTestCase.java
index 596baed7..d14d284d 100644
--- a/src/test/java/org/apache/commons/beanutils2/memoryleaktests/MemoryLeakTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/memoryleaktests/MemoryLeakTestCase.java
@@ -361,9 +361,8 @@ public class MemoryLeakTestCase {
// The classLoader will go away only when these following variables are released
ClassLoader loader = newClassLoader();
- Class<?> beanClass = loader.loadClass(className);
- Object bean = beanClass.newInstance();
-
+ Class<?> beanClass = loader.loadClass(className);
+ Object bean = beanClass.newInstance();
final WeakReference<ClassLoader> someRef = new WeakReference<>(loader);
@@ -375,13 +374,13 @@ public class MemoryLeakTestCase {
// if you comment the following two lines, the test will work, and the ClassLoader will be released.
// That proves that nothing is wrong with the test, and ConvertUtilsBean is holding a reference
- ConvertUtils.register(new IntegerConverter(), beanClass);
+ ConvertUtils.register(new IntegerConverter(), (Class<Integer>) beanClass);
assertEquals("12345", ConvertUtils.convert(bean, String.class));
// this should make the reference go away.
- loader = null;
+ loader = null;
beanClass = null;
- bean = null;
+ bean = null;
forceGarbageCollection(); /* Try to force the garbage collector to run by filling up memory */
[commons-beanutils] 03/06: Rename test class to match class under test
Posted by gg...@apache.org.
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-beanutils.git
commit eed83986f166793ca8b79e42f173bd956b52b962
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Feb 9 10:05:12 2023 -0500
Rename test class to match class under test
---
.../{BeanUtils2TestCase.java => BeanUtilsBean2TestCase.java} | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanUtilsBean2TestCase.java
similarity index 97%
rename from src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java
rename to src/test/java/org/apache/commons/beanutils2/BeanUtilsBean2TestCase.java
index 7680f046..cb4baa51 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanUtils2TestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanUtilsBean2TestCase.java
@@ -23,21 +23,17 @@ import junit.framework.TestSuite;
* Test Case for the {@link BeanUtilsBean2}.
*
*/
-public class BeanUtils2TestCase extends BeanUtilsBeanTestCase {
-
-
+public class BeanUtilsBean2TestCase extends BeanUtilsBeanTestCase {
/**
* Constructs a new instance of this test case.
*
* @param name Name of the test case
*/
- public BeanUtils2TestCase(final String name) {
+ public BeanUtilsBean2TestCase(final String name) {
super(name);
}
-
-
/**
* Sets up instance variables required by this test case.
*/
@@ -52,7 +48,7 @@ public class BeanUtils2TestCase extends BeanUtilsBeanTestCase {
* Creates the tests included in this test suite.
*/
public static Test suite() {
- return new TestSuite(BeanUtils2TestCase.class);
+ return new TestSuite(BeanUtilsBean2TestCase.class);
}
/**
[commons-beanutils] 04/06: Remove extra whitespace
Posted by gg...@apache.org.
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-beanutils.git
commit 2cd4830b586d366264b245e08f0c23b962853033
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Feb 9 10:06:15 2023 -0500
Remove extra whitespace
---
.../apache/commons/beanutils2/BeanUtilsBenchCase.java | 17 -----------------
1 file changed, 17 deletions(-)
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanUtilsBenchCase.java b/src/test/java/org/apache/commons/beanutils2/BeanUtilsBenchCase.java
index 4b36bf55..783f5756 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanUtilsBenchCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanUtilsBenchCase.java
@@ -29,24 +29,17 @@ import junit.framework.TestSuite;
* JUnit Test Case containing microbenchmarks for BeanUtils.
*
*/
-
public class BeanUtilsBenchCase extends TestCase {
-
-
/**
* Constructs a new instance of this test case.
*
* @param name Name of the test case
*/
public BeanUtilsBenchCase(final String name) {
-
super(name);
-
}
-
-
// Basic loop counter
private long counter = 100000;
@@ -66,8 +59,6 @@ public class BeanUtilsBenchCase extends TestCase {
// BeanUtilsBean instance to be used
private BeanUtilsBean bu = null;
-
-
/**
* Sets up instance variables required by this test case.
*/
@@ -136,9 +127,7 @@ public class BeanUtilsBenchCase extends TestCase {
* Creates the tests included in this test suite.
*/
public static Test suite() {
-
return new TestSuite(BeanUtilsBenchCase.class);
-
}
/**
@@ -146,7 +135,6 @@ public class BeanUtilsBenchCase extends TestCase {
*/
@Override
public void tearDown() {
-
dynaClass = null;
inBean = null;
inDyna = null;
@@ -154,11 +142,8 @@ public class BeanUtilsBenchCase extends TestCase {
outBean = null;
outDyna = null;
bu = null;
-
}
-
-
// Time copyProperties() from a bean
public void testCopyPropertiesBean() throws Exception {
@@ -352,6 +337,4 @@ public class BeanUtilsBenchCase extends TestCase {
}
-
-
}
[commons-beanutils] 05/06: Generics
Posted by gg...@apache.org.
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-beanutils.git
commit e833b2c52ca71fc6d29ae1196cd2788869be3fde
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Feb 9 10:32:33 2023 -0500
Generics
---
src/main/java/org/apache/commons/beanutils2/ConvertUtils.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/main/java/org/apache/commons/beanutils2/ConvertUtils.java b/src/main/java/org/apache/commons/beanutils2/ConvertUtils.java
index 4e255377..319b436d 100644
--- a/src/main/java/org/apache/commons/beanutils2/ConvertUtils.java
+++ b/src/main/java/org/apache/commons/beanutils2/ConvertUtils.java
@@ -125,7 +125,7 @@ public class ConvertUtils {
* @return The registered {@link Converter} or {@code null} if not found
* @see ConvertUtilsBean#lookup(Class)
*/
- public static Converter lookup(final Class<?> clazz) {
+ public static <T> Converter<T> lookup(final Class<T> clazz) {
return ConvertUtilsBean.getInstance().lookup(clazz);
}
@@ -138,7 +138,7 @@ public class ConvertUtils {
* @param targetType Class of the value to be converted to
* @return The registered {@link Converter} or {@code null} if not found
*/
- public static Converter lookup(final Class<?> sourceType, final Class<?> targetType) {
+ public static <T> Converter<T> lookup(final Class<?> sourceType, final Class<T> targetType) {
return ConvertUtilsBean.getInstance().lookup(sourceType, targetType);
}
@@ -153,7 +153,7 @@ public class ConvertUtils {
* Converter
* @see ConvertUtilsBean#register(Converter, Class)
*/
- public static void register(final Converter converter, final Class<?> clazz) {
+ public static <T> void register(final Converter<T> converter, final Class<T> clazz) {
ConvertUtilsBean.getInstance().register(converter, clazz);
}
[commons-beanutils] 01/06: Remove extra whitespace
Posted by gg...@apache.org.
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-beanutils.git
commit b6bdccad0abf3763b5354fead57c3aab18cdd972
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Feb 9 10:03:16 2023 -0500
Remove extra whitespace
---
src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java
index c09ec9e8..e1adeed8 100644
--- a/src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils2/BeanUtilsTestCase.java
@@ -60,8 +60,6 @@ import junit.framework.TestSuite;
*/
public class BeanUtilsTestCase extends TestCase {
-
-
/**
* The test bean for each test.
*/