DevFW-CICD/spring-petclinic
16
0
Fork 1
mirror of https://github.com/spring-projects/spring-petclinic.git synced 2025-07-19 05:55:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix test method

Browse source 
Test method was not working. Should be working now.
This commit is contained in:
Zackkogan 2018-04-02 23:33:09 -04:00
parent fc12751b2c
commit ab3082021f
1 changed files with 28 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

39
src/test/java/org/springframework/samples/petclinic/newDataStore/NewOwnerStoreTest.java
View file

@ -7,13 +7,12 @@ package org.springframework.samples.petclinic.newDataStore;
import java.util.*;
import java.util.regex.Pattern;
import java.util.Iterator;
import java.util.Map;
import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
import org.springframework.samples.petclinic.owner.Owner;
@ -22,6 +21,8 @@ import org.springframework.samples.petclinic.owner.StaticOwner;
import org.springframework.test.context.junit4.SpringRunner;
import static org.junit.Assert.*;
import static org.mockito.BDDMockito.given;
import static org.mockito.Mockito.doReturn;
/**
* @author Gibran
@ -30,22 +31,30 @@ import static org.junit.Assert.*;
@DataJpaTest
public class NewOwnerStoreTest {
private static final int TEST_OWNER_ID = 1;
@Autowired
OwnerRepository owners;
OwnerRepository ownerRepository;
NewOwnerStore testOwnerStore;
HashMap<Integer, StaticOwner> ownerStore;
Map<Integer, StaticOwner> ownerStore;
@Mock
Owner mockOwner;
Owner owner;
@Before
public void setup() {
testOwnerStore = NewOwnerStore.getInstance(owners);
testOwnerStore = NewOwnerStore.getInstance(ownerRepository);
testOwnerStore.forklift();
ownerStore = testOwnerStore.getStore();
MockitoAnnotations.initMocks(this);
doReturn(TEST_OWNER_ID).when(owner).getId();
doReturn("John").when(owner).getFirstName();
doReturn("Doe").when(owner).getLastName();
doReturn("123 Cucumber Lane").when(owner).getAddress();
doReturn("Placeville").when(owner).getCity();
doReturn("1234567890").when(owner).getTelephone();
given(this.ownerRepository.findById(TEST_OWNER_ID)).willReturn(owner);
}
@Test
@ -60,10 +69,10 @@ public class NewOwnerStoreTest {
@Test
public void testShadowRead() {
testOwnerStore = NewOwnerStore.getInstance(owners);
testOwnerStore = NewOwnerStore.getInstance(ownerRepository);
testOwnerStore.forklift();
Collection<Owner> results = this.owners.findByLastName("");
Collection<Owner> results = this.ownerRepository.findByLastName("");
int inconsistencies = compareResults(results, "");
@ -116,16 +125,16 @@ public class NewOwnerStoreTest {
}
@Test
public void checkShadowWrite() {
testOwnerStore = NewOwnerStore.getInstance(owners);
public void testShadowWrite() {
testOwnerStore = NewOwnerStore.getInstance(ownerRepository);
//make sure that inconsisties are recorded and fixed
testOwnerStore.testPutInOldDatastoreOnly(mockOwner);
//make sure that inconsistencies are recorded and fixed
testOwnerStore.testPutInOldDatastoreOnly(owner);
assertEquals(1, testOwnerStore.checkConsistency());
assertEquals(0, testOwnerStore.checkConsistency());
//make sure that any changes written to old database are also written to new database
testOwnerStore.save(mockOwner);
testOwnerStore.save(owner);
assertEquals(0, testOwnerStore.checkConsistency());
}
}