minor improvement in synchronization unit tests

gbellato

03.37.2006 567b227b63c4a65f9e4c457763109cd646cc2476

minor improvement in synchronization unit tests 

 opends/tests/unit-tests-testng/src/server/org/opends/server/synchronization/ChangeNumberTest.java

@@ -244,13 +244,13 @@

    // Generate a changeNumber separates by 10 milliseconds
    // and check that they are differents
    Thread.currentThread().sleep(10);
    Thread.sleep(10);
    CN2 = cng.NewChangeNumber();
    assertTrue(CN1.compareTo(CN2) != 0);

    // Generate a changeNumber separates by 300 milliseconds
    // and check that they are differents
    Thread.currentThread().sleep(300);
    Thread.sleep(300);
    CN2 = cng.NewChangeNumber();
    assertTrue(CN1.compareTo(CN2) != 0);


 opends/tests/unit-tests-testng/src/server/org/opends/server/synchronization/ModifyConflictTest.java

@@ -161,7 +161,7 @@

    // Get the historical uuid associated to the entry
    // (the one that needs to be tested)
    String uuid = hist.getEntryUuid(entry);
    String uuid = Historical.getEntryUuid(entry);

    // Get the Entry uuid in String format
    List<Attribute> uuidAttrs = entry
@@ -178,16 +178,6 @@
      }
    }
    
    try
    {
      Historical dup = hist.duplicate();
      // TODO Check values
    }
    catch (RuntimeException e)
    {
      assertTrue(false) ;
    }
    
    
    // Test FakeOperation
    try
@@ -224,7 +214,7 @@

    // Get the historical uuid associated to the entry
    // (the one that needs to be tested)
    String uuid = hist.getEntryUuid(addOp);
    String uuid = Historical.getEntryUuid(addOp);

    // Get the op uuid in String format
    List<Attribute> uuidAttrs = addOp.getOperationalAttributes().get(
@@ -297,6 +287,7 @@
    ArrayList<Attribute> histList = new ArrayList<Attribute>(1);
    Attribute histAttr = new Attribute(Historical.historicalAttrType,
        "ds-sync-hist", valuesHist);
    histList.add(0, histAttr);

    //Add the historical att in the entry
    entry.putAttribute(Historical.historicalAttrType,histList) ;

 opends/tests/unit-tests-testng/src/server/org/opends/server/synchronization/ServerStateTest.java

@@ -26,41 +26,14 @@
 */
package org.opends.server.synchronization;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.LinkedHashSet;
import java.util.List;
import java.util.Set;
import java.util.Map;
import java.util.zip.DataFormatException;

import org.testng.annotations.DataProvider;
import org.testng.annotations.Test;
import static org.testng.Assert.*;

import org.opends.server.api.ClientConnection;
import org.opends.server.core.AddOperation;
import org.opends.server.core.DeleteOperation;
import org.opends.server.core.DirectoryServer;
import org.opends.server.core.ModifyDNOperation;
import org.opends.server.core.ModifyOperation;
import org.opends.server.core.Operation;
import org.opends.server.protocols.internal.InternalClientConnection;
import org.opends.server.protocols.ldap.LDAPAttribute;
import org.opends.server.synchronization.ModifyMsg;
import org.opends.server.types.Attribute;
import org.opends.server.types.AttributeType;
import org.opends.server.types.AttributeValue;
import org.opends.server.types.Control;
import org.opends.server.types.DirectoryException;
import org.opends.server.types.DN;
import org.opends.server.types.Modification;
import org.opends.server.types.ModificationType;
import org.opends.server.types.ObjectClass;
import org.opends.server.types.RDN;
import org.opends.server.util.TimeThread;
import java.util.Set;

import static org.opends.server.synchronization.OperationContext.*;
import org.opends.server.types.DN;
import org.opends.server.util.TimeThread;
import org.testng.annotations.DataProvider;
import org.testng.annotations.Test;

/**
 * Test the ServerState
@@ -96,7 +69,8 @@

    // Check getServerStateDn()
    DN returned_DN = serverState.getServerStateDn();
    // TODO Check the returned DN
    // Check that the returned DN stays below dn
    assertTrue(dn.isAncestorOf(returned_DN));

    // Check update
    assertFalse(serverState.update(null));
@@ -123,7 +97,9 @@

    // Check the toString
    String stringRep = serverState.toString() ;
    // TODO Check the value
    assertFalse(stringRep.isEmpty());
    assertTrue(stringRep.contains(cn2.toString()));
    assertTrue(stringRep.contains(cn3.toString()));

    // Check getBytes
    byte[] b = serverState.getBytes();

 opends/tests/unit-tests-testng/src/server/org/opends/server/synchronization/SynchronizationMsgTest.java

@@ -174,7 +174,6 @@
         throws Exception
  {
    DN dn = DN.decode(rawdn);
    InternalClientConnection connection = new InternalClientConnection();
    ModifyMsg msg = new ModifyMsg(changeNumber, dn, mods, "fakeuniqueid");;

    // Check uuid

 opends/tests/unit-tests-testng/src/server/org/opends/server/synchronization/UpdateOperationTest.java

@@ -35,7 +35,6 @@
import org.testng.annotations.BeforeClass;
import org.testng.annotations.Test;

import static org.opends.server.synchronization.OperationContext.SYNCHROCONTEXT;
import static org.testng.Assert.*;

import org.opends.server.TestCaseUtils;