dhis2-devs team mailing list archive

[noreply@xxxxxxxxxxxxx]

------------------------------------------------------------
revno: 4630
committer: Morten Olav Hansen <mortenoh@xxxxxxxxx>
branch nick: dhis2
timestamp: Wed 2011-09-21 17:53:54 +0200
message:
  fixed unit tests for attributevalue{service|store}
modified:
  dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueServiceTest.java
  dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueStoreTest.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/attribute/AttributeValueServiceTest.java'
--- dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueServiceTest.java	2011-09-07 08:03:18 +0000
+++ dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueServiceTest.java	2011-09-21 15:53:54 +0000
@@ -36,21 +36,34 @@
 {
     private AttributeService attributeService;
 
+    private AttributeValue attributeValue1;
+
+    private AttributeValue attributeValue2;
+    
     @Override
     protected void setUpTest()
     {
         attributeService = (AttributeService) getBean( "org.hisp.dhis.attribute.AttributeService" );
+
+        attributeValue1 = new AttributeValue( "value 1" );
+        attributeValue2 = new AttributeValue( "value 2" );
+
+        Attribute attribute1 = new Attribute( "attribute 1", "string" );
+        Attribute attribute2 = new Attribute( "attribute 2", "string" );
+
+        attributeService.addAttribute( attribute1 );
+        attributeService.addAttribute( attribute2 );
+
+        attributeValue1.setAttribute( attribute1 );
+        attributeValue2.setAttribute( attribute2 );
+
+        attributeService.addAttributeValue( attributeValue1 );
+        attributeService.addAttributeValue( attributeValue2 );
     }
 
     @Test
     public void testAddAttributeValue()
     {
-        AttributeValue attributeValue1 = new AttributeValue( "value 1" );
-        AttributeValue attributeValue2 = new AttributeValue( "value 2" );
-
-        attributeService.addAttributeValue( attributeValue1 );
-        attributeService.addAttributeValue( attributeValue2 );
-
         attributeValue1 = attributeService.getAttributeValue( attributeValue1.getId() );
         attributeValue2 = attributeService.getAttributeValue( attributeValue2.getId() );
 
@@ -61,12 +74,6 @@
     @Test
     public void testUpdateAttributeValue()
     {
-        AttributeValue attributeValue1 = new AttributeValue( "value 1" );
-        AttributeValue attributeValue2 = new AttributeValue( "value 2" );
-
-        attributeService.addAttributeValue( attributeValue1 );
-        attributeService.addAttributeValue( attributeValue2 );
-
         attributeValue1.setValue( "updated value 1" );
         attributeValue2.setValue( "updated value 2" );
 
@@ -86,12 +93,6 @@
     @Test
     public void testDeleteAttributeValue()
     {
-        AttributeValue attributeValue1 = new AttributeValue( "value 1" );
-        AttributeValue attributeValue2 = new AttributeValue( "value 2" );
-
-        attributeService.addAttributeValue( attributeValue1 );
-        attributeService.addAttributeValue( attributeValue2 );
-
         int attributeValueId1 = attributeValue1.getId();
         int attributeValueId2 = attributeValue2.getId();
 
@@ -108,12 +109,6 @@
     @Test
     public void testGetAttributeValue()
     {
-        AttributeValue attributeValue1 = new AttributeValue( "value 1" );
-        AttributeValue attributeValue2 = new AttributeValue( "value 2" );
-
-        attributeService.addAttributeValue( attributeValue1 );
-        attributeService.addAttributeValue( attributeValue2 );
-
         attributeValue1 = attributeService.getAttributeValue( attributeValue1.getId() );
         attributeValue2 = attributeService.getAttributeValue( attributeValue2.getId() );
 
@@ -124,12 +119,6 @@
     @Test
     public void testGetAllAttributeValues()
     {
-        AttributeValue attributeValue1 = new AttributeValue( "value 1" );
-        AttributeValue attributeValue2 = new AttributeValue( "value 2" );
-
-        attributeService.addAttributeValue( attributeValue1 );
-        attributeService.addAttributeValue( attributeValue2 );
-
         assertEquals( 2, attributeService.getAllAttributeValues().size() );
     }
 }

=== modified file 'dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueStoreTest.java'
--- dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueStoreTest.java	2011-09-06 11:52:18 +0000
+++ dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/attribute/AttributeValueStoreTest.java	2011-09-21 15:53:54 +0000
@@ -40,16 +40,19 @@
     @Override
     protected void setUpTest()
     {
+        AttributeStore attributeStore = (AttributeStore) getBean( "org.hisp.dhis.attribute.AttributeStore" );
         attributeValueStore = (AttributeValueStore) getBean( "org.hisp.dhis.attribute.AttributeValueStore" );
 
         Attribute attribute1 = new Attribute();
         attribute1.setName( "attribute_simple" );
         attribute1.setValueType( "string" );
 
-        attributeValue1 = new AttributeValue("value 1");
+        attributeStore.save( attribute1 );
+
+        attributeValue1 = new AttributeValue( "value 1" );
         attributeValue1.setAttribute( attribute1 );
 
-        attributeValue2 = new AttributeValue("value 2");
+        attributeValue2 = new AttributeValue( "value 2" );
         attributeValue2.setAttribute( attribute1 );
 
         attributeValueStore.save( attributeValue1 );