From 57d38c450aab6379ce7c08d58757cdddba7ef1ab Mon Sep 17 00:00:00 2001
From: matthew_swift <matthew_swift@localhost>
Date: Mon, 02 Oct 2006 09:39:35 +0000
Subject: [PATCH] Add test suite for org.opends.server.types.ObjectClass and perform the following refactoring:

---
 opends/src/server/org/opends/server/core/DirectoryServer.java |   66 ++++++++++----------------------
 1 files changed, 21 insertions(+), 45 deletions(-)

diff --git a/opends/src/server/org/opends/server/core/DirectoryServer.java b/opends/src/server/org/opends/server/core/DirectoryServer.java
index 4dfa929..de68210 100644
--- a/opends/src/server/org/opends/server/core/DirectoryServer.java
+++ b/opends/src/server/org/opends/server/core/DirectoryServer.java
@@ -2867,24 +2867,10 @@
          directoryServer.schema.getObjectClass(TOP_OBJECTCLASS_NAME);
     if (objectClass == null)
     {
-      ConcurrentHashMap<String,String> names =
-           new ConcurrentHashMap<String,String>(1);
-      names.put(TOP_OBJECTCLASS_NAME, TOP_OBJECTCLASS_NAME);
-
-      CopyOnWriteArraySet<AttributeType> requiredAttrs =
-           new CopyOnWriteArraySet<AttributeType>();
-      CopyOnWriteArraySet<AttributeType> optionalAttrs =
-           new CopyOnWriteArraySet<AttributeType>();
-
-      ConcurrentHashMap<String,CopyOnWriteArrayList<String>> extraProperties =
-           new ConcurrentHashMap<String,CopyOnWriteArrayList<String>>(0);
-
-      objectClass = new ObjectClass(TOP_OBJECTCLASS_NAME, names,
-                                    TOP_OBJECTCLASS_OID,
-                                    TOP_OBJECTCLASS_DESCRIPTION, null,
-                                    requiredAttrs, optionalAttrs,
-                                    ObjectClassType.ABSTRACT, false,
-                                    extraProperties);
+      objectClass = new ObjectClass(TOP_OBJECTCLASS_NAME, Collections
+          .singleton(TOP_OBJECTCLASS_NAME), TOP_OBJECTCLASS_OID,
+          TOP_OBJECTCLASS_DESCRIPTION, null, null, null,
+          ObjectClassType.ABSTRACT, false, null);
     }
 
     return objectClass;
@@ -2893,14 +2879,16 @@
 
 
   /**
-   * Causes the Directory Server to construct a new objectclass definition with
-   * the provided name and with no required or allowed attributes.  This should
-   * only be used if there is no objectclass for the specified name.  It will
-   * not register the created objectclass with the Directory Server.
+   * Causes the Directory Server to construct a new objectclass
+   * definition with the provided name and with no required or allowed
+   * attributes. This should only be used if there is no objectclass
+   * for the specified name. It will not register the created
+   * objectclass with the Directory Server.
    *
-   * @param  name  The name to use for the objectclass, as provided by the user.
-   *
-   * @return  The constructed objectclass definition.
+   * @param name
+   *          The name to use for the objectclass, as provided by the
+   *          user.
+   * @return The constructed objectclass definition.
    */
   public static ObjectClass getDefaultObjectClass(String name)
   {
@@ -2911,22 +2899,10 @@
     ObjectClass objectClass = directoryServer.schema.getObjectClass(lowerName);
     if (objectClass == null)
     {
-      ConcurrentHashMap<String,String> names =
-           new ConcurrentHashMap<String,String>(1);
-      names.put(lowerName, name);
-
-      CopyOnWriteArraySet<AttributeType> requiredAttrs =
-           new CopyOnWriteArraySet<AttributeType>();
-      CopyOnWriteArraySet<AttributeType> optionalAttrs =
-           new CopyOnWriteArraySet<AttributeType>();
-
-      ConcurrentHashMap<String,CopyOnWriteArrayList<String>> extraProperties =
-           new ConcurrentHashMap<String,CopyOnWriteArrayList<String>>(0);
-
-      objectClass = new ObjectClass(name, names, lowerName, null,
-                                    getTopObjectClass(), requiredAttrs,
-                                    optionalAttrs, ObjectClassType.ABSTRACT,
-                                    false, extraProperties);
+      objectClass = new ObjectClass(name,
+          Collections.singleton(name), lowerName, null,
+          getTopObjectClass(), null, null, ObjectClassType.ABSTRACT,
+          false, null);
     }
 
     return objectClass;
@@ -2935,11 +2911,11 @@
 
 
   /**
-   * Retrieves the set of attribute type definitions that have been defined in
-   * the Directory Server.
+   * Retrieves the set of attribute type definitions that have been
+   * defined in the Directory Server.
    *
-   * @return  The set of attribute type definitions that have been defined in
-   *          the Directory Server.
+   * @return The set of attribute type definitions that have been
+   *         defined in the Directory Server.
    */
   public static ConcurrentHashMap<String,AttributeType> getAttributeTypes()
   {

--
Gitblit v1.10.0