Polishing

This commit is contained in:
Juergen Hoeller
2014-11-29 20:49:21 +01:00
parent bfbd25a0e9
commit efb114d49a
3 changed files with 9 additions and 7 deletions

View File

@@ -61,7 +61,7 @@ public interface ConfigurablePropertyAccessor extends PropertyAccessor, Property
* <p>If {@code true}, a {@code null} path location will be populated * <p>If {@code true}, a {@code null} path location will be populated
* with a default object value and traversed instead of resulting in a * with a default object value and traversed instead of resulting in a
* {@link NullValueInNestedPathException}. * {@link NullValueInNestedPathException}.
* <p>Default is {@code false} on a plain instance. * <p>Default is {@code false} on a plain PropertyAccessor instance.
*/ */
void setAutoGrowNestedPaths(boolean autoGrowNestedPaths); void setAutoGrowNestedPaths(boolean autoGrowNestedPaths);

View File

@@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2013 the original author or authors. * Copyright 2002-2014 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@@ -31,15 +31,17 @@ import static org.junit.Assert.*;
*/ */
public class BeanWrapperAutoGrowingTests { public class BeanWrapperAutoGrowingTests {
Bean bean = new Bean(); private final Bean bean = new Bean();
private final BeanWrapperImpl wrapper = new BeanWrapperImpl(bean);
BeanWrapperImpl wrapper = new BeanWrapperImpl(bean);
@Before @Before
public void setUp() { public void setUp() {
wrapper.setAutoGrowNestedPaths(true); wrapper.setAutoGrowNestedPaths(true);
} }
@Test @Test
public void getPropertyValueNullValueInNestedPath() { public void getPropertyValueNullValueInNestedPath() {
assertNull(wrapper.getPropertyValue("nested.prop")); assertNull(wrapper.getPropertyValue("nested.prop"));
@@ -51,7 +53,7 @@ public class BeanWrapperAutoGrowingTests {
assertEquals("test", bean.getNested().getProp()); assertEquals("test", bean.getNested().getProp());
} }
@Test(expected=NullValueInNestedPathException.class) @Test(expected = NullValueInNestedPathException.class)
public void getPropertyValueNullValueInNestedPathNoDefaultConstructor() { public void getPropertyValueNullValueInNestedPathNoDefaultConstructor() {
wrapper.getPropertyValue("nestedNoConstructor.prop"); wrapper.getPropertyValue("nestedNoConstructor.prop");
} }
@@ -139,7 +141,7 @@ public class BeanWrapperAutoGrowingTests {
assertThat(bean.getMultiList().get(0).get(0), instanceOf(Bean.class)); assertThat(bean.getMultiList().get(0).get(0), instanceOf(Bean.class));
} }
@Test(expected=InvalidPropertyException.class) @Test(expected = InvalidPropertyException.class)
public void getPropertyValueAutoGrowListNotParameterized() { public void getPropertyValueAutoGrowListNotParameterized() {
wrapper.getPropertyValue("listNotParameterized[0]"); wrapper.getPropertyValue("listNotParameterized[0]");
} }

View File

@@ -141,7 +141,7 @@ public class TypeDescriptor implements Serializable {
* <p>Returns primitive types as-is. * <p>Returns primitive types as-is.
* <p>See {@link #getObjectType()} for a variation of this operation that * <p>See {@link #getObjectType()} for a variation of this operation that
* resolves primitive types to their corresponding Object types if necessary. * resolves primitive types to their corresponding Object types if necessary.
* @return the type, or {@code null} * @return the type, or {@code null} if it cannot be determined
* @see #getObjectType() * @see #getObjectType()
*/ */
public Class<?> getType() { public Class<?> getType() {