Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@

import org.socialsignin.spring.data.dynamodb.core.DynamoDBOperations;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;

Expand All @@ -28,6 +29,7 @@ public AbstractSingleEntityQuery(DynamoDBOperations dynamoDBOperations, Class<T>

@Override
public List<T> getResultList() {
return Arrays.asList(getSingleResult());
T result = getSingleResult();
return result != null ? Arrays.asList(result) : new ArrayList<>();
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
*/
package org.socialsignin.spring.data.dynamodb.query;

import org.junit.Before;
import org.junit.Test;
import org.mockito.Mock;
import org.socialsignin.spring.data.dynamodb.core.DynamoDBOperations;
Expand All @@ -29,27 +28,36 @@ public class AbstractSingleEntityQueryTest {

@Mock
private DynamoDBOperations dynamoDBOperations;
@Mock
private User entity;

private final User entity = new User();

private AbstractSingleEntityQuery<User> underTest;

@Before
public void setUp() {
@Test
public void testGetResultList() {
underTest = new AbstractSingleEntityQuery<User>(dynamoDBOperations, User.class) {
@Override
public User getSingleResult() {
return entity;
}
};
}

@Test
public void testGetResultList() {
List<User> actual = underTest.getResultList();

assertEquals(1, actual.size());
assertEquals(entity, actual.get(0));
}

@Test
public void testGetResultListEmpty() {
underTest = new AbstractSingleEntityQuery<User>(dynamoDBOperations, User.class) {
@Override
public User getSingleResult() { return null; }
};

List<User> actual = underTest.getResultList();

assertEquals(0, actual.size());
}

}