renamed executeConversion to simply convert for readibility
This commit is contained in:
@@ -18,7 +18,7 @@ package org.springframework.core.convert;
|
||||
/**
|
||||
* A service interface for type conversion. This is the entry point into the convert system.
|
||||
* <p>
|
||||
* Call {@link #executeConversion(Object, TypeDescriptor)} to perform a thread-safe type conversion using
|
||||
* Call {@link #convert(Object, TypeDescriptor)} to perform a thread-safe type conversion using
|
||||
* this system.
|
||||
*
|
||||
* @author Keith Donald
|
||||
@@ -43,6 +43,6 @@ public interface ConversionService {
|
||||
* source to an instance of targetType
|
||||
* @throws ConversionException if an exception occurred during the conversion process
|
||||
*/
|
||||
public Object executeConversion(Object source, TypeDescriptor targetType);
|
||||
public Object convert(Object source, TypeDescriptor targetType);
|
||||
|
||||
}
|
||||
@@ -140,7 +140,7 @@ public class GenericConversionService implements ConversionService {
|
||||
}
|
||||
}
|
||||
|
||||
public Object executeConversion(Object source, TypeDescriptor targetType) {
|
||||
public Object convert(Object source, TypeDescriptor targetType) {
|
||||
if (source == null) {
|
||||
return null;
|
||||
}
|
||||
@@ -149,7 +149,7 @@ public class GenericConversionService implements ConversionService {
|
||||
return executor.execute(source);
|
||||
} else {
|
||||
if (parent != null) {
|
||||
return parent.executeConversion(source, targetType);
|
||||
return parent.convert(source, targetType);
|
||||
} else {
|
||||
throw new ConverterNotFoundException(source.getClass(), targetType,
|
||||
"No converter found that can convert from sourceType [" + source.getClass().getName()
|
||||
|
||||
@@ -45,17 +45,17 @@ public class GenericConversionServiceTests {
|
||||
@Test
|
||||
public void executeConversion() {
|
||||
service.addConverter(new StringToInteger());
|
||||
assertEquals(new Integer(3), service.executeConversion("3", type(Integer.class)));
|
||||
assertEquals(new Integer(3), service.convert("3", type(Integer.class)));
|
||||
}
|
||||
|
||||
@Test
|
||||
public void executeConversionNullSource() {
|
||||
assertEquals(null, service.executeConversion(null, type(Integer.class)));
|
||||
assertEquals(null, service.convert(null, type(Integer.class)));
|
||||
}
|
||||
|
||||
@Test
|
||||
public void executeCompatibleSource() {
|
||||
assertEquals(false, service.executeConversion(false, type(boolean.class)));
|
||||
assertEquals(false, service.convert(false, type(boolean.class)));
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -83,7 +83,7 @@ public class GenericConversionServiceTests {
|
||||
@Test
|
||||
public void convertExecutorNotFound() {
|
||||
try {
|
||||
service.executeConversion("3", type(Integer.class));
|
||||
service.convert("3", type(Integer.class));
|
||||
fail("Should have thrown an exception");
|
||||
} catch (ConverterNotFoundException e) {
|
||||
}
|
||||
@@ -160,7 +160,7 @@ public class GenericConversionServiceTests {
|
||||
}
|
||||
});
|
||||
try {
|
||||
service.executeConversion("3", type(Integer.class));
|
||||
service.convert("3", type(Integer.class));
|
||||
fail("Should have failed");
|
||||
} catch (ConverterNotFoundException e) {
|
||||
|
||||
@@ -271,7 +271,7 @@ public class GenericConversionServiceTests {
|
||||
foo.put("2", "BAZ");
|
||||
service.addConverter(new StringToInteger());
|
||||
service.addConverter(new StringToEnum());
|
||||
service.executeConversion(foo, new TypeDescriptor(getClass().getField("genericMap")));
|
||||
service.convert(foo, new TypeDescriptor(getClass().getField("genericMap")));
|
||||
}
|
||||
|
||||
@Ignore
|
||||
@@ -359,7 +359,7 @@ public class GenericConversionServiceTests {
|
||||
@Test
|
||||
public void testSuperTwoWayConverterConverterAdaption() {
|
||||
service.addConverter(GenericConversionService.converterFor(String.class, FooEnum.class, new StringToEnum()));
|
||||
assertEquals(FooEnum.BAR, service.executeConversion("BAR", type(FooEnum.class)));
|
||||
assertEquals(FooEnum.BAR, service.convert("BAR", type(FooEnum.class)));
|
||||
}
|
||||
|
||||
private TypeDescriptor type(Class<?> clazz) {
|
||||
|
||||
Reference in New Issue
Block a user