=== modified file 'dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/period/PeriodServiceTest.java' --- dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/period/PeriodServiceTest.java 2014-03-24 22:27:13 +0000 +++ dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/period/PeriodServiceTest.java 2014-04-27 22:59:30 +0000 @@ -701,4 +701,31 @@ assertNotNull( periodTypeB ); assertEquals( refB.getName(), periodTypeB.getName() ); } + + @Test + public void testDeleteAndGetPeriodType() + throws Exception + { + Collection periodTypes = periodService.getAllPeriodTypes(); + Iterator it = periodTypes.iterator(); + PeriodType periodTypeA = it.next(); + PeriodType periodTypeB = it.next(); + PeriodType periodTypeC = it.next(); + PeriodType periodTypeD = it.next(); + + int idA = periodTypeA.getId(); + int idB = periodTypeB.getId(); + int idC = periodTypeC.getId(); + int idD = periodTypeD.getId(); + + assertNotNull( periodService.getPeriodType( idA ) ); + assertNotNull( periodService.getPeriodType( idB ) ); + assertNotNull( periodService.getPeriodType( idC ) ); + assertNotNull( periodService.getPeriodType( idD ) ); + + assertNotNull( periodService.getPeriodType( periodTypeA.getId() ) ); + assertNotNull( periodService.getPeriodType( periodTypeB.getId() ) ); + assertNotNull( periodService.getPeriodType( periodTypeC.getId() ) ); + assertNotNull( periodService.getPeriodType( periodTypeD.getId() ) ); + } } === modified file 'dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/period/PeriodStoreTest.java' --- dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/period/PeriodStoreTest.java 2014-04-27 22:31:59 +0000 +++ dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/period/PeriodStoreTest.java 2014-04-27 22:59:30 +0000 @@ -83,7 +83,7 @@ optionCombo = categoryService.getDefaultDataElementCategoryOptionCombo(); } - + // ------------------------------------------------------------------------- // Period // ------------------------------------------------------------------------- @@ -624,79 +624,4 @@ periods = periodStore.getPeriods( mar, dataElements2, sources2 ); } - - // ------------------------------------------------------------------------- - // PeriodType - // ------------------------------------------------------------------------- - - @Test - public void testDeleteAndGetPeriodType() - throws Exception - { - Collection periodTypes = periodStore.getAllPeriodTypes(); - Iterator it = periodTypes.iterator(); - PeriodType periodTypeA = it.next(); - PeriodType periodTypeB = it.next(); - PeriodType periodTypeC = it.next(); - PeriodType periodTypeD = it.next(); - - int idA = periodTypeA.getId(); - int idB = periodTypeB.getId(); - int idC = periodTypeC.getId(); - int idD = periodTypeD.getId(); - - assertNotNull( periodStore.getPeriodType( idA ) ); - assertNotNull( periodStore.getPeriodType( idB ) ); - assertNotNull( periodStore.getPeriodType( idC ) ); - assertNotNull( periodStore.getPeriodType( idD ) ); - - assertNotNull( periodStore.getPeriodType( periodTypeA.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeB.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeC.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeD.getClass() ) ); - - periodStore.deletePeriodType( periodTypeA ); - assertNull( periodStore.getPeriodType( idA ) ); - assertNotNull( periodStore.getPeriodType( idB ) ); - assertNotNull( periodStore.getPeriodType( idC ) ); - assertNotNull( periodStore.getPeriodType( idD ) ); - - assertNull( periodStore.getPeriodType( periodTypeA.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeB.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeC.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeD.getClass() ) ); - - periodStore.deletePeriodType( periodTypeB ); - assertNull( periodStore.getPeriodType( idA ) ); - assertNull( periodStore.getPeriodType( idB ) ); - assertNotNull( periodStore.getPeriodType( idC ) ); - assertNotNull( periodStore.getPeriodType( idD ) ); - - assertNull( periodStore.getPeriodType( periodTypeA.getClass() ) ); - assertNull( periodStore.getPeriodType( periodTypeB.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeC.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeD.getClass() ) ); - - periodStore.deletePeriodType( periodTypeC ); - assertNull( periodStore.getPeriodType( idA ) ); - assertNull( periodStore.getPeriodType( idB ) ); - assertNull( periodStore.getPeriodType( idC ) ); - assertNotNull( periodStore.getPeriodType( idD ) ); - - assertNull( periodStore.getPeriodType( periodTypeA.getClass() ) ); - assertNull( periodStore.getPeriodType( periodTypeB.getClass() ) ); - assertNull( periodStore.getPeriodType( periodTypeC.getClass() ) ); - assertNotNull( periodStore.getPeriodType( periodTypeD.getClass() ) ); - - periodStore.deletePeriodType( periodTypeD ); - assertNull( periodStore.getPeriodType( idA ) ); - assertNull( periodStore.getPeriodType( idB ) ); - assertNull( periodStore.getPeriodType( idC ) ); - assertNull( periodStore.getPeriodType( idD ) ); - - assertNull( periodStore.getPeriodType( periodTypeA.getClass() ) ); - assertNull( periodStore.getPeriodType( periodTypeB.getClass() ) ); - assertNull( periodStore.getPeriodType( periodTypeC.getClass() ) ); - assertNull( periodStore.getPeriodType( periodTypeD.getClass() ) ); - } }