Skip to content

Commit b625cae

Browse files
committed
[test] Restore the getName as no real reason to remove
yes its not used, neither is the internal object name. If cleanup keeps running it will just remove more and more which makes its usage unclear. Add comment instead
1 parent 998b614 commit b625cae

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

src/test/java/org/mybatis/spring/batch/MyBatisCursorItemReaderTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,12 +90,17 @@ private List<Object> getFoos() {
9090
return Arrays.asList(new Foo("foo1"), new Foo("foo2"), new Foo("foo3"));
9191
}
9292

93+
// Note: Do not cleanup this 'foo' class
9394
private static class Foo {
9495
private final String name;
9596

9697
Foo(String name) {
9798
this.name = name;
9899
}
100+
101+
public String getName() {
102+
return this.name;
103+
}
99104
}
100105

101106
}

src/test/java/org/mybatis/spring/batch/builder/MyBatisBatchItemWriterBuilderTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -176,12 +176,17 @@ private Chunk<Foo> getFoos() {
176176
return Chunk.of(new Foo("foo1"), new Foo("foo2"), new Foo("foo3"));
177177
}
178178

179+
// Note: Do not cleanup this 'foo' class
179180
private static class Foo {
180181
private final String name;
181182

182183
Foo(String name) {
183184
this.name = name;
184185
}
186+
187+
public String getName() {
188+
return this.name;
189+
}
185190
}
186191

187192
}

0 commit comments

Comments
 (0)