Skip to content

Commit 8638a21

Browse files
authored
Merge pull request #347 from domaframework/issue-345
Fix #345
2 parents 8ef216a + 89b30cc commit 8638a21

File tree

2 files changed

+23
-6
lines changed

2 files changed

+23
-6
lines changed

src/main/java/org/seasar/doma/internal/jdbc/sql/ScalarResultListParameter.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import static org.seasar.doma.internal.util.AssertionUtil.assertNotNull;
44

55
import java.util.ArrayList;
6-
import java.util.List;
76
import java.util.function.Supplier;
87
import org.seasar.doma.internal.jdbc.command.ScalarProvider;
98
import org.seasar.doma.internal.jdbc.scalar.Scalar;
@@ -25,9 +24,4 @@ public ScalarResultListParameter(Supplier<Scalar<BASIC, CONTAINER>> supplier) {
2524
public ObjectProvider<CONTAINER> createObjectProvider(Query query) {
2625
return new ScalarProvider<>(supplier, query);
2726
}
28-
29-
@Override
30-
public List<CONTAINER> getResult() {
31-
return null;
32-
}
3327
}
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
package org.seasar.doma.internal.jdbc.sql;
2+
3+
import static org.junit.jupiter.api.Assertions.assertEquals;
4+
5+
import java.util.List;
6+
import org.junit.jupiter.api.Test;
7+
import org.seasar.doma.internal.jdbc.scalar.BasicScalar;
8+
import org.seasar.doma.wrapper.IntegerWrapper;
9+
10+
class ScalarResultListParameterTest {
11+
12+
@Test
13+
void getResult() {
14+
ScalarResultListParameter<Integer, Integer> parameter =
15+
new ScalarResultListParameter<>(() -> new BasicScalar<>(IntegerWrapper::new));
16+
parameter.add(1);
17+
parameter.add(2);
18+
List<Integer> result = parameter.getResult();
19+
assertEquals(2, result.size());
20+
assertEquals(1, result.get(0));
21+
assertEquals(2, result.get(1));
22+
}
23+
}

0 commit comments

Comments
 (0)