[jboss-svn-commits] JBL Code SVN: r23365 - in labs/jbossrules/trunk/drools-process/drools-process-task/src: test/java/org/drools/userprofile and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: jervisliu
Date: 2008-10-08 03:28:16 -0400 (Wed, 08 Oct 2008)
New Revision: 23365

Modified:
   labs/jbossrules/trunk/drools-process/drools-process-task/src/main/java/org/drools/userprofile/UserProfileManager.java
   labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/MockFileBasedUserProfileRepository.java
   labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/UserProfileManagerTest.java
Log:
minor updates to UserProfileManager interface.

Modified: labs/jbossrules/trunk/drools-process/drools-process-task/src/main/java/org/drools/userprofile/UserProfileManager.java
===================================================================
--- labs/jbossrules/trunk/drools-process/drools-process-task/src/main/java/org/drools/userprofile/UserProfileManager.java	2008-10-08 07:13:07 UTC (rev 23364)
+++ labs/jbossrules/trunk/drools-process/drools-process-task/src/main/java/org/drools/userprofile/UserProfileManager.java	2008-10-08 07:28:16 UTC (rev 23365)
@@ -47,8 +47,12 @@
 		return user;
 	}
 	
-	public void updateUserProfile(UserProfile info) {
-				
+	public void updateUser(User user) {
+		if (userProfileRepository == null) {
+			//TODO: throws exception?
+			return;
+		}
+		userProfileRepository.setUserProfile(user.getUserProfile());			
 	}
 	
 	public List<User> getUsers() {

Modified: labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/MockFileBasedUserProfileRepository.java
===================================================================
--- labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/MockFileBasedUserProfileRepository.java	2008-10-08 07:13:07 UTC (rev 23364)
+++ labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/MockFileBasedUserProfileRepository.java	2008-10-08 07:28:16 UTC (rev 23365)
@@ -1,5 +1,6 @@
 package org.drools.userprofile;
 
+import java.util.ArrayList;
 import java.util.List;
 
 public class MockFileBasedUserProfileRepository implements UserProfileRepository {
@@ -12,15 +13,29 @@
 	}
 
 	public void setUserProfile(UserProfile info) {
-		//NOT IMPLEMENTED;
+		if(!(info instanceof DroolsTaskUserProfile)) {
+			return;
+		}
+		DroolsTaskUserProfile dtup = (DroolsTaskUserProfile)info;
+		//update file properties
 	}
 	
 	
     public List<User> getUsers() {
+    	//may need to look into RMDB to get the list of all users
 		return null;
 	}
 	
 	public List<Group> getGroups() {
+		List<OrganizationalEntity> members = new ArrayList<OrganizationalEntity>();
+		members.add(new User());
+		
+		Group group = new Group();
+		group.setMembers(members);
+
+		List<Group> result = new ArrayList<Group>();
+		
+		result.add(group);
 		return null;
     }
 }

Modified: labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/UserProfileManagerTest.java
===================================================================
--- labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/UserProfileManagerTest.java	2008-10-08 07:13:07 UTC (rev 23364)
+++ labs/jbossrules/trunk/drools-process/drools-process-task/src/test/java/org/drools/userprofile/UserProfileManagerTest.java	2008-10-08 07:28:16 UTC (rev 23365)
@@ -31,8 +31,10 @@
     	upm.setUserProfileRepository(new MockFileBasedUserProfileRepository());
 
     	User user = (User)upm.getUser();
+    	assertEquals(user.getId(), "mockedUser");
+
     	DroolsTaskUserProfile userProfile = (DroolsTaskUserProfile)user.getUserProfile();
-    	assertEquals(userProfile.getID(), "mockedUser");
+    	//assertEquals(userProfile.getDisplayName(entity), "mockedUserName");
     }
 
 }