[gatein-commits] gatein SVN: r1087 - in portal/trunk/component/identity/src: test/java/org/exoplatform/services/organization and 1 other directory.

Author: trong.tran Date: 2009-12-21 03:54:31 -0500 (Mon, 21 Dec 2009) New Revision: 1087 Modified: portal/trunk/component/identity/src/main/java/org/exoplatform/services/organization/idm/GroupDAOImpl.java portal/trunk/component/identity/src/test/java/org/exoplatform/services/organization/TestOrganizationService.java Log: GTNPORTAL-373 Update unit test for testing Group Modified: portal/trunk/component/identity/src/main/java/org/exoplatform/services/organization/idm/GroupDAOImpl.java =================================================================== --- portal/trunk/component/identity/src/main/java/org/exoplatform/services/organization/idm/GroupDAOImpl.java 2009-12-21 06:26:46 UTC (rev 1086) +++ portal/trunk/component/identity/src/main/java/org/exoplatform/services/organization/idm/GroupDAOImpl.java 2009-12-21 08:54:31 UTC (rev 1087) @@ -429,7 +429,6 @@ attrs = attrsList.toArray(attrs); - //getIdentitySession().getAttributesManager().addAttributes(jbidGroup, attrs); getIdentitySession().getAttributesManager().updateAttributes(jbidGroup, attrs); } Modified: portal/trunk/component/identity/src/test/java/org/exoplatform/services/organization/TestOrganizationService.java =================================================================== --- portal/trunk/component/identity/src/test/java/org/exoplatform/services/organization/TestOrganizationService.java 2009-12-21 06:26:46 UTC (rev 1086) +++ portal/trunk/component/identity/src/test/java/org/exoplatform/services/organization/TestOrganizationService.java 2009-12-21 08:54:31 UTC (rev 1087) @@ -198,15 +198,21 @@ /* Create a child group with name: Group1 */ Group groupChild1 = groupHandler_.createGroupInstance(); groupChild1.setGroupName(Group1); + groupChild1.setLabel("Group1 Label"); groupHandler_.addChild(groupParent, groupChild1, true); + + assertEquals(groupHandler_.findGroupById(groupChild1.getId()).getLabel(), "Group1 Label"); + groupChild1 = groupHandler_.findGroupById(groupChild1.getId()); assertEquals(groupChild1.getParentId(), groupParent.getId()); assertEquals("Expect group child's name is: ", Group1, groupChild1.getGroupName()); /* Update groupChild's information */ - groupChild1.setLabel("GroupRenamed"); + groupChild1.setLabel("Group1 Label renamed"); groupChild1.setDescription("new description "); groupHandler_.saveGroup(groupChild1, true); - assertEquals(groupHandler_.findGroupById(groupChild1.getId()).getLabel(), "GroupRenamed"); + + assertEquals(groupHandler_.findGroupById(groupChild1.getId()).getLabel(), "Group1 Label renamed"); + /* Create a group child with name is: Group2 */ Group groupChild2 = groupHandler_.createGroupInstance(); groupChild2.setGroupName(Group2);