dhis2-devs team mailing list archive

[noreply@xxxxxxxxxxxxx]

------------------------------------------------------------
revno: 15036
committer: Lars Helge Overland <larshelge@xxxxxxxxx>
branch nick: dhis2
timestamp: Mon 2014-04-28 00:59:30 +0200
message:
  Test fix
modified:
  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/PeriodStoreTest.java


--
lp:dhis2
https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk

Your team DHIS 2 developers is subscribed to branch lp:dhis2.
To unsubscribe from this branch go to https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk/+edit-subscription

=== 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<PeriodType> periodTypes = periodService.getAllPeriodTypes();
+        Iterator<PeriodType> 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<PeriodType> periodTypes = periodStore.getAllPeriodTypes();
-        Iterator<PeriodType> 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() ) );
-    }
 }