Fixed some minor issues that were pointed out in review of the messages com...

kenneth_suter

17.35.2007 3e8f6ef36b4d8bacc1470bfebcacda7b6e2fc351

Fixed some minor issues that were pointed out in review of the messages commit.  This biggest issues here is the reversion of the Validator class to use String based messages instead of the message framework as the class is only used internally.

 opends/src/guitools/org/opends/guitools/statuspanel/StatusPanelController.java

 opends/src/quicksetup/org/opends/quicksetup/ui/WebBrowserErrorDialog.java

@@ -189,7 +189,7 @@
      // UIFactory.initialize();
      WebBrowserErrorDialog dlg =
          new WebBrowserErrorDialog(new JFrame(),
              new WebBrowserException("http://www.yahoo.com",
              new WebBrowserException("http://opends.org",
                      Message.raw("toto"), null));
      dlg.packAndShow();
    } catch (Exception ex)

 opends/src/server/org/opends/server/core/PasswordPolicy.java

 opends/src/server/org/opends/server/types/ErrorLogSeverity.java

File was deleted

 opends/src/server/org/opends/server/util/Validator.java

@@ -31,9 +31,6 @@
import org.opends.server.loggers.debug.DebugTracer;
import org.opends.server.loggers.ErrorLogger;


import org.opends.messages.MessageBuilder;

/**
 * This utility class provides static methods that make parameter checking
 * easier (e.g. in constructors and setters).
@@ -154,7 +151,7 @@
  public static boolean ensureNotNull(Object param)
          throws AssertionError {
    if (ENABLE_CHECKS) {
      if (param == null) throwNull(Message.EMPTY);
      if (param == null) throwNull("");
    }
    return true;
  }
@@ -186,8 +183,8 @@
  public static boolean ensureNotNull(Object param1, Object param2)
          throws AssertionError {
    if (ENABLE_CHECKS) {
      if (param1 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[1]));
      if (param2 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[2]));
      if (param1 == null) throwNull(PARAM_DESCRIPTIONS[1]);
      if (param2 == null) throwNull(PARAM_DESCRIPTIONS[2]);
    }
    return true;
  }
@@ -221,9 +218,9 @@
                                      Object param3)
          throws AssertionError {
    if (ENABLE_CHECKS) {
      if (param1 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[1]));
      if (param2 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[2]));
      if (param3 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[3]));
      if (param1 == null) throwNull(PARAM_DESCRIPTIONS[1]);
      if (param2 == null) throwNull(PARAM_DESCRIPTIONS[2]);
      if (param3 == null) throwNull(PARAM_DESCRIPTIONS[3]);
    }
    return true;
  }
@@ -258,10 +255,10 @@
                                      Object param3, Object param4)
          throws AssertionError {
    if (ENABLE_CHECKS) {
      if (param1 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[1]));
      if (param2 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[2]));
      if (param3 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[3]));
      if (param4 == null) throwNull(Message.raw(PARAM_DESCRIPTIONS[4]));
      if (param1 == null) throwNull(PARAM_DESCRIPTIONS[1]);
      if (param2 == null) throwNull(PARAM_DESCRIPTIONS[2]);
      if (param3 == null) throwNull(PARAM_DESCRIPTIONS[3]);
      if (param4 == null) throwNull(PARAM_DESCRIPTIONS[4]);
    }
    return true;
  }
@@ -289,7 +286,7 @@
          throws AssertionError {
    if (ENABLE_CHECKS) {
      if (!condition) {
        ensureTrue(condition, Message.EMPTY);
        ensureTrue(condition, "");
      }
    }
    return true;
@@ -317,18 +314,18 @@
   *
   * @throws AssertionError if condition is false
   */
  public static boolean ensureTrue(boolean condition, Message message)
  public static boolean ensureTrue(boolean condition, String message)
          throws AssertionError {
    if (ENABLE_CHECKS) {
      if (!condition) {
        MessageBuilder mb = new MessageBuilder();
        StringBuilder mb = new StringBuilder();
        mb.append("The specified condition must be true. ");
        mb.append(message);
        Message fullMessage = generateLineSpecificErrorMessage(mb.toMessage());
        String fullString = generateLineSpecificErrorString(mb.toString());

        logError(fullMessage);
        logError(fullString);

        throw new AssertionError(fullMessage);
        throw new AssertionError(fullString);
      }
    }
    return true;
@@ -377,37 +374,37 @@
  ////////////////////////////////////////////////////////////////////////////


  private static Message generateLineSpecificErrorMessage(Message message) {
    MessageBuilder mb = new MessageBuilder();
  private static String generateLineSpecificErrorString(String message) {
    StringBuilder mb = new StringBuilder();
    mb.append(message);
    mb.append("  The error occurred at ");
    mb.append(getOriginalCallerLineInfo());
    return mb.toMessage();
    return mb.toString();
  }


  private static void throwNull(Message message)
  private static void throwNull(String message)
          throws AssertionError {
    MessageBuilder mb = new MessageBuilder();
    StringBuilder mb = new StringBuilder();
    mb.append("The specified parameter must not be null. ");
    mb.append(message);
    Message fullMessage = generateLineSpecificErrorMessage(mb.toMessage());
    String fullString = generateLineSpecificErrorString(mb.toString());

    logError(fullMessage);
    logError(fullString);

    throw new AssertionError(fullMessage);
    throw new AssertionError(fullString);
  }



  private static void logError(Message message) {
  private static void logError(String message) {
    incrementErrorCount();

    MessageBuilder mb = new MessageBuilder();
    StringBuilder mb = new StringBuilder();
    mb.append(message);
    mb.append(ServerConstants.EOL);
    mb.append(getCallingStack());
    Message messageWithStack = mb.toMessage();
    String messageWithStack = mb.toString();

    // Log to the debug log.
    if (debugEnabled())
@@ -416,7 +413,7 @@
    }

    // Log to the error log.
    ErrorLogger.logError(messageWithStack);
    ErrorLogger.logError(Message.raw(messageWithStack));
  }



 opends/tests/unit-tests-testng/src/server/org/opends/server/util/ValidatorTests.java

@@ -75,7 +75,7 @@

  @Test
  public void testEnsureTrueWithMessage() {
    boolean returnValue = Validator.ensureTrue(true, Message.raw("some message"));
    boolean returnValue = Validator.ensureTrue(true, "some message");
    assertTrue(returnValue);  // must always return true
  }

@@ -144,20 +144,20 @@

  @Test(expectedExceptions = {RuntimeException.class, AssertionError.class})
  public void testEnsureTrueWithMessageWithFalse() {
    Validator.ensureTrue(false, Message.raw("some message"));
    Validator.ensureTrue(false, "some message");
  }


  @Test
  public void testMessageContents() {
    Validator.resetErrorCount();
    Message myMessage = Message.raw("some test message");
    String myMessage = "some test message";
    String thisMethod = ValidatorTests.class.getName() + "." + "testMessageContents(ValidatorTests.java:";
    try {
      Validator.ensureTrue(false, myMessage);
    } catch (Throwable e) {
      String caughtMessage = e.getMessage();
      assertTrue(caughtMessage.indexOf(myMessage.toString()) >= 0);
      assertTrue(caughtMessage.indexOf(myMessage) >= 0);
      assertTrue(caughtMessage.indexOf(thisMethod) >= 0);

      assertEquals(Validator.getErrorCount(), 1);