mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

lutoff
03.01.2008 a8be62629a05cca6809657e76ae587381da71f76
Fix for issue #2594 (list-backends CLI should not return 0 when basedn has a wrong value)

When a provided valid DN id neither a based DN nor below a based DN, we return "1"
2 files modified
6 ■■■■■ changed files
opends/src/server/org/opends/server/tools/ListBackends.java 4 ●●● patch | view | raw | blame | history
opends/tests/unit-tests-testng/src/server/org/opends/server/tools/ListBackendsTestCase.java 2 ●●● patch | view | raw | blame | history
opends/src/server/org/opends/server/tools/ListBackends.java
@@ -336,6 +336,7 @@
    // backend ID argument was present, then list the base DNs for that backend.
    // If the base DN argument was present, then list the backend for that base
    // DN.  If no arguments were provided, then list all backends and base DNs.
    boolean invalidDn = false;
    if (baseDN.isPresent())
    {
      // Create a map from the base DNs of the backends to the corresponding
@@ -391,6 +392,7 @@
              message = INFO_LISTBACKENDS_NO_BACKEND_FOR_DN.get(
                      dn.toString());
              out.println(message);
              invalidDn = true;
              break;
            }
            else
@@ -505,7 +507,7 @@
    // If we've gotten here, then everything completed successfully.
    return 0;
    return invalidDn ? 1 : 0 ;
  }
opends/tests/unit-tests-testng/src/server/org/opends/server/tools/ListBackendsTestCase.java
@@ -357,7 +357,7 @@
      "--baseDN", "ou=nonexistent"
    };
    assertEquals(ListBackends.listBackends(args, false, null, null), 0);
    assertEquals(ListBackends.listBackends(args, false, null, null), 1);
  }