updated coding rules.

[utils] / security / src / test / java / org / wamblee / usermgt / InMemoryGroupSetTest.java

diff --git a/security/src/test/java/org/wamblee/usermgt/InMemoryGroupSetTest.java b/security/src/test/java/org/wamblee/usermgt/InMemoryGroupSetTest.java
index 2ea119058425fee83056b9b924d35c96f7e28eeb..03329d1d958f918b150e220e424ad54d3c6d043a 100644 (file)
--- a/security/src/test/java/org/wamblee/usermgt/InMemoryGroupSetTest.java
+++ b/security/src/test/java/org/wamblee/usermgt/InMemoryGroupSetTest.java
@@ -27,7 +27,7 @@ import junit.framework.TestCase;
  */
 public class InMemoryGroupSetTest extends TestCase {
     
-    protected GroupSet _groups; 
+    protected GroupSet groups; 
    
     /**
      * This method must be overriden in subclasses. 
@@ -43,7 +43,7 @@ public class InMemoryGroupSetTest extends TestCase {
     @Override
     protected void setUp() throws Exception {
         super.setUp();
-        _groups = createGroupSet(); 
+        groups = createGroupSet(); 
         checkGroupCount(0); 
     }
     
@@ -70,7 +70,7 @@ public class InMemoryGroupSetTest extends TestCase {
      * @param aSize Expected number of groups. 
      */
     protected void checkGroupCount(int aSize) throws SQLException { 
-        assertEquals(aSize, _groups.size());
+        assertEquals(aSize, groups.size());
     }
     
     /**
@@ -80,13 +80,13 @@ public class InMemoryGroupSetTest extends TestCase {
      */
     public void testAdd() throws SQLException { 
        Group group = new Group("group1"); 
-       assertTrue(  _groups.add(group) );
+       assertTrue(  groups.add(group) );
        checkGroupExists(group.getName());
        checkGroupCount(1);
-       Group group2 = _groups.find("group1"); 
+       Group group2 = groups.find("group1"); 
        assertNotNull(group2);
        assertEquals(group.getName(), group2.getName()); 
-       Set<Group> set = _groups.list(); 
+       Set<Group> set = groups.list(); 
        assertEquals(1, set.size()); 
        assertTrue(set.contains(group));
     }
@@ -99,12 +99,12 @@ public class InMemoryGroupSetTest extends TestCase {
     public void testFindUnknownGroup() throws SQLException { 
        Group group1 = new Group("group1"); 
        Group group2 = new Group("group2");
-       _groups.add(group1); 
-       _groups.add(group2);
+       groups.add(group1); 
+       groups.add(group2);
        checkGroupExists(group1.getName()); 
        checkGroupExists(group2.getName()); 
     
-       assertNull( _groups.find("group3") );
+       assertNull( groups.find("group3") );
        checkGroupNotExists("group3");
     }
     
@@ -113,13 +113,13 @@ public class InMemoryGroupSetTest extends TestCase {
      */
     public void testAddDuplicateGroup() throws SQLException { 
        Group group1 = new Group("group1"); 
-       _groups.add(group1); 
+       groups.add(group1); 
        
-       assertEquals(1, _groups.list().size()); 
-       assertTrue(_groups.contains(group1));
+       assertEquals(1, groups.list().size()); 
+       assertTrue(groups.contains(group1));
        group1 = new Group("group1");
-       assertFalse(_groups.add(group1));
-       assertEquals(1, _groups.list().size());
+       assertFalse(groups.add(group1));
+       assertEquals(1, groups.list().size());
        
        checkGroupExists(group1.getName()); 
        checkGroupCount(1); 
@@ -132,14 +132,14 @@ public class InMemoryGroupSetTest extends TestCase {
      */
     public void testRemoveGroup() throws SQLException { 
         Group group1 = new Group("group1"); 
-        _groups.add(group1); 
-        assertTrue(_groups.contains(group1));
+        groups.add(group1); 
+        assertTrue(groups.contains(group1));
         checkGroupCount(1);
         
-        assertTrue(_groups.remove(group1)); 
-        assertFalse(_groups.contains(group1)); 
-        assertNull(_groups.find(group1.getName()));
-        assertEquals(0, _groups.list().size());
+        assertTrue(groups.remove(group1)); 
+        assertFalse(groups.contains(group1)); 
+        assertNull(groups.find(group1.getName()));
+        assertEquals(0, groups.list().size());
         checkGroupCount(0);
     }
   
@@ -150,14 +150,14 @@ public class InMemoryGroupSetTest extends TestCase {
      */
     public void testRemoveNonExistingGroup() throws SQLException { 
         Group group1 = new Group("group1"); 
-        _groups.add(group1); 
+        groups.add(group1); 
         checkGroupCount(1);
         Group nonExistingGroup = new Group("group2"); 
         nonExistingGroup.setPrimaryKey(new Long(1000));
         nonExistingGroup.setPersistedVersion(1000);
-        assertFalse(_groups.remove(nonExistingGroup));
-        assertTrue(_groups.contains(group1)); 
-        assertEquals(1, _groups.list().size());
+        assertFalse(groups.remove(nonExistingGroup));
+        assertTrue(groups.contains(group1)); 
+        assertEquals(1, groups.list().size());
         checkGroupCount(1); 
     }
     
@@ -170,15 +170,15 @@ public class InMemoryGroupSetTest extends TestCase {
         Group group1 = new Group("group1"); 
         Group group2 = new Group("group2"); 
         Group group3 = new Group("group3"); 
-        assertTrue(_groups.add(group1)); 
-        assertTrue(_groups.add(group2)); 
-        assertTrue(_groups.add(group3));
+        assertTrue(groups.add(group1)); 
+        assertTrue(groups.add(group2)); 
+        assertTrue(groups.add(group3));
         
         checkGroupExists(group1.getName()); 
         checkGroupExists(group2.getName()); 
         checkGroupExists(group3.getName()); 
         
-        Set<Group> set = _groups.list(); 
+        Set<Group> set = groups.list(); 
         assertTrue(set.contains(group1)); 
         assertTrue(set.contains(group2)); 
         assertTrue(set.contains(group3));