Skip to content

Commit ecbfc5c

Browse files
gsmetgunnarmorling
authored andcommitted
HV-1471 Minor cleanup of unused code in a test
1 parent a5988f4 commit ecbfc5c

File tree

1 file changed

+2
-32
lines changed
  • engine/src/test/java/org/hibernate/validator/test/internal/engine/valueextraction

1 file changed

+2
-32
lines changed

engine/src/test/java/org/hibernate/validator/test/internal/engine/valueextraction/UnwrappingTest.java

Lines changed: 2 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@
2626
import javax.validation.Validator;
2727
import javax.validation.constraints.Future;
2828
import javax.validation.constraints.Min;
29-
import javax.validation.constraints.NotBlank;
30-
import javax.validation.constraints.Null;
3129
import javax.validation.valueextraction.ExtractedValue;
3230
import javax.validation.valueextraction.UnwrapByDefault;
3331
import javax.validation.valueextraction.Unwrapping;
@@ -45,7 +43,6 @@
4543
* @author Guillaume Smet
4644
*/
4745
@TestForIssue(jiraKey = "HV-925")
48-
@SuppressWarnings("unused")
4946
public class UnwrappingTest {
5047
private Validator validatorWithValueExtractor;
5148
private Validator validatorWithoutValueExtractor;
@@ -109,41 +106,12 @@ private class Foobar {
109106
private final ValueHolder<Integer> integerHolder = new ValueHolder<>( 5 );
110107
}
111108

112-
private class Bar {
113-
@Min(10)
114-
private final ValueHolder<Integer> integerHolder = new ValueHolder<>( 5 );
115-
116-
private final ValueHolder<@NotBlank String> stringHolder = new ValueHolder<>( "" );
117-
}
118-
119-
private class Baz {
120-
@Null
121-
private final ValueHolder<Integer> integerHolder = new ValueHolder<>( 5 );
122-
}
123-
124-
private class Qux {
125-
@ValueHolderConstraint
126-
private final ValueHolder<Integer> integerHolder = new ValueHolder<>( 5 );
127-
}
128-
129-
private class WrapperWithImplicitUnwrapping {
130-
131-
@Min(10)
132-
private final Wrapper<Integer> integerWrapper = new Wrapper<>( 5 );
133-
}
134-
135109
private class WrapperWithDisabledUnwrapping {
136110

137111
@Min(value = 10, payload = { Unwrapping.Skip.class })
138112
private final Wrapper<Integer> integerWrapper = new Wrapper<>( 5 );
139113
}
140114

141-
private class WrapperWithForcedUnwrapping {
142-
143-
@Min(value = 10, payload = { Unwrapping.Unwrap.class })
144-
private final Wrapper<Integer> integerWrapper = new Wrapper<>( 5 );
145-
}
146-
147115
private class BeanWithWrapperWithTwoTypeArguments {
148116

149117
@Min(value = 10)
@@ -158,6 +126,7 @@ private ValueHolder(T value) {
158126
this.value = value;
159127
}
160128

129+
@SuppressWarnings("unused")
161130
public T getValue() {
162131
return value;
163132
}
@@ -171,6 +140,7 @@ private Wrapper(T value) {
171140
this.value = value;
172141
}
173142

143+
@SuppressWarnings("unused")
174144
public T getValue() {
175145
return value;
176146
}

0 commit comments

Comments
 (0)