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

Yannick Lecaillez
18.51.2015 74bc0fc87005bb2971da0957c287729f6806d384
Code cleanup
7 files modified
133 ■■■■ changed files
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/ControlsTestCase.java 6 ●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/EntryIDSetTest.java 66 ●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/ID2CountTest.java 5 ●●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/PluggableBackendImplTestCase.java 18 ●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/StateTest.java 4 ●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/TestDnKeyFormat.java 28 ●●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/Utils.java 6 ●●●● patch | view | raw | blame | history
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/ControlsTestCase.java
@@ -598,7 +598,7 @@
  private ArrayList<DN> getDNs(final LinkedList<SearchResultEntry> entries)
  {
    final ArrayList<DN> results = new ArrayList<DN>();
    final ArrayList<DN> results = new ArrayList<>();
    for (final Entry e : entries)
    {
      results.add(e.getName());
@@ -608,7 +608,7 @@
  private List<DN> getDNs(List<Integer> expectedOrder) throws Exception
  {
    List<DN> dns = new ArrayList<DN>(expectedOrder.size());
    List<DN> dns = new ArrayList<>(expectedOrder.size());
    for (int i : expectedOrder)
    {
      dns.add(USERS[i].toDN());
@@ -739,7 +739,7 @@
    private Entry toEntry() throws Exception
    {
      final List<String> ldif = new ArrayList<String>();
      final List<String> ldif = new ArrayList<>();
      ldif.add("dn: " + toDN());
      ldif.add("objectClass: top");
      ldif.add("objectClass: person");
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/EntryIDSetTest.java
@@ -50,25 +50,25 @@
  @Test
  public void testDefinedAdd()
  {
    EntryIDSet set = newDefinedSet(6, 8, 10, 12);
    final EntryIDSet set = newDefinedSet(6, 8, 10, 12);
    assertThat(set.add(new EntryID(4))).isTrue();
    assertIdsEquals(set, 4L, 6, 8, 10, 12);
    assertThat(set.add(id(4))).isTrue();
    assertIdsEquals(set, 4, 6, 8, 10, 12);
    assertThat(set.add(new EntryID(14L))).isTrue();
    assertIdsEquals(set, 4L, 6, 8, 10, 12, 14L);
    assertThat(set.add(id(14))).isTrue();
    assertIdsEquals(set, 4, 6, 8, 10, 12, 14);
    assertThat(set.add(new EntryID(11))).isTrue();
    assertIdsEquals(set, 4L, 6, 8, 10, 11, 12, 14L);
    assertThat(set.add(id(11))).isTrue();
    assertIdsEquals(set, 4, 6, 8, 10, 11, 12, 14);
    assertThat(set.add(new EntryID(10))).isFalse();
    assertIdsEquals(set, 4L, 6, 8, 10, 11, 12, 14);
    assertThat(set.add(id(10))).isFalse();
    assertIdsEquals(set, 4, 6, 8, 10, 11, 12, 14);
  }
  @Test
  public void testDefinedAddAll()
  {
    EntryIDSet set = newDefinedSet(10, 12);
    final EntryIDSet set = newDefinedSet(10, 12);
    // Add nothing
    set.addAll(newDefinedSet());
@@ -98,25 +98,25 @@
  @Test
  public void testDefinedRemove()
  {
    EntryIDSet set = newDefinedSet(4, 6, 8, 10, 12, 14);
    final EntryIDSet set = newDefinedSet(4, 6, 8, 10, 12, 14);
    assertThat(set.remove(new EntryID(4))).isTrue();
    assertThat(set.remove(id(4))).isTrue();
    assertIdsEquals(set, 6, 8, 10, 12, 14);
    assertThat(set.remove(new EntryID(14))).isTrue();
    assertThat(set.remove(id(14))).isTrue();
    assertIdsEquals(set, 6, 8, 10, 12);
    assertThat(set.remove(new EntryID(10))).isTrue();
    assertThat(set.remove(id(10))).isTrue();
    assertIdsEquals(set, 6, 8, 12);
    assertThat(set.remove(new EntryID(10))).isFalse();
    assertThat(set.remove(id(10))).isFalse();
    assertIdsEquals(set, 6, 8, 12);
  }
  @Test
  public void testDefinedRemoveAll()
  {
    EntryIDSet set = newDefinedSet(1, 2, 4, 6, 8, 9, 10, 12, 13, 14, 16, 18, 20, 21);
    final EntryIDSet set = newDefinedSet(1, 2, 4, 6, 8, 9, 10, 12, 13, 14, 16, 18, 20, 21);
    // Remove nothing
    set.removeAll(newDefinedSet());
@@ -142,33 +142,33 @@
  @Test
  public void testDefinedContain()
  {
    EntryIDSet set = newDefinedSet(4, 6, 8, 10, 12, 14);
    final EntryIDSet set = newDefinedSet(4, 6, 8, 10, 12, 14);
    assertThat(set.contains(new EntryID(2))).isFalse();
    assertThat(set.contains(new EntryID(4))).isTrue();
    assertThat(set.contains(id(2))).isFalse();
    assertThat(set.contains(id(4))).isTrue();
    assertThat(set.contains(new EntryID(9))).isFalse();
    assertThat(set.contains(new EntryID(10))).isTrue();
    assertThat(set.contains(id(9))).isFalse();
    assertThat(set.contains(id(10))).isTrue();
    assertThat(set.contains(new EntryID(14))).isTrue();
    assertThat(set.contains(new EntryID(16))).isFalse();
    assertThat(set.contains(id(14))).isTrue();
    assertThat(set.contains(id(16))).isFalse();
  }
  @Test
  public void testDefinedIterator()
  {
    assertIdsEquals(newDefinedSet(4, 6, 8, 10, 12).iterator(), 4L, 6L, 8L, 10L, 12L);
    assertIdsEquals(newDefinedSet(4, 6, 8, 10, 12).iterator(), 4, 6, 8, 10, 12);
  }
  @Test
  public void testDefinedIteratorWithBegin()
  {
    EntryIDSet set = newDefinedSet(4, 6, 8, 10, 12);
    final EntryIDSet set = newDefinedSet(4, 6, 8, 10, 12);
    assertIdsEquals(set.iterator(new EntryID(4)), 4L, 6L, 8L, 10L, 12L);
    assertIdsEquals(set.iterator(new EntryID(8)), 8L, 10L, 12L);
    assertIdsEquals(set.iterator(new EntryID(12)), 12L);
    assertIdsEquals(set.iterator(new EntryID(13)), 4L, 6L, 8L, 10L, 12L);
    assertIdsEquals(set.iterator(id(4)), 4, 6, 8, 10, 12);
    assertIdsEquals(set.iterator(id(8)), 8, 10, 12);
    assertIdsEquals(set.iterator(id(12)), 12);
    assertIdsEquals(set.iterator(id(13)), 4, 6, 8, 10, 12);
  }
  @Test(dataProvider = "codecs")
@@ -208,7 +208,7 @@
  public void testUndefinedAddDoesNothing()
  {
    final EntryIDSet undefined = newUndefinedSet();
    assertThat(undefined.add(new EntryID(4))).isTrue();
    assertThat(undefined.add(id(4))).isTrue();
    assertThat(undefined.size()).isEqualTo(Long.MAX_VALUE);
  }
@@ -228,7 +228,7 @@
  public void testUndefinedRemoveDoesNothing()
  {
    final EntryIDSet undefined = newUndefinedSet();
    assertThat(undefined.remove(new EntryID(4))).isTrue();
    assertThat(undefined.remove(id(4))).isTrue();
    assertThat(undefined.size()).isEqualTo(Long.MAX_VALUE);
  }
@@ -243,7 +243,7 @@
  @Test
  public void testUndefinedContain()
  {
    assertThat(newUndefinedSet().contains(new EntryID(4))).isTrue();
    assertThat(newUndefinedSet().contains(id(4))).isTrue();
  }
  @Test
@@ -255,7 +255,7 @@
  @Test
  public void testUndefinedIteratorWithBegin()
  {
    assertThat(newUndefinedSet().iterator(new EntryID(8)).hasNext()).isFalse();
    assertThat(newUndefinedSet().iterator(id(8)).hasNext()).isFalse();
  }
  @Test
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/ID2CountTest.java
@@ -35,7 +35,6 @@
import java.util.concurrent.TimeUnit;
import org.forgerock.opendj.config.server.ConfigException;
import org.forgerock.opendj.ldap.ByteString;
import org.forgerock.util.promise.NeverThrowsException;
import org.forgerock.util.promise.PromiseImpl;
import org.opends.server.DirectoryServerTestCase;
@@ -87,9 +86,9 @@
    when(serverContext.getDiskSpaceMonitor()).thenReturn(mock(DiskSpaceMonitor.class));
    storage = new PersistItStorage(createBackendCfg(), serverContext);
    org.opends.server.backends.pluggable.spi.Importer importer = storage.startImport();
    try(final org.opends.server.backends.pluggable.spi.Importer importer = storage.startImport()) {
    importer.createTree(id2CountTreeName);
    importer.close();
    }
    storage.open();
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/PluggableBackendImplTestCase.java
@@ -750,7 +750,7 @@
  private List<Entry> getDbEntries(List<Entry> entries) throws DirectoryException
  {
    List<Entry> result = new ArrayList<Entry>(entries.size());
    List<Entry> result = new ArrayList<>(entries.size());
    for (Entry currentEntry : entries)
    {
      Entry dbEntry = backend.getEntry(currentEntry.getName());
@@ -840,16 +840,13 @@
    ByteArrayOutputStream rejectedEntries = new ByteArrayOutputStream();
    LDIFImportConfig importConf = new LDIFImportConfig(templateFile);
    try (final LDIFImportConfig importConf = new LDIFImportConfig(templateFile))
    {
    importConf.setInvokeImportPlugins(true);
    importConf.setClearBackend(true);
    importConf.writeRejectedEntries(rejectedEntries);
    try {
      backend.importLDIF(importConf, DirectoryServer.getInstance().getServerContext());
    }
    finally {
      importConf.close();
    }
    assertEquals(rejectedEntries.size(), 0, "No entries should be rejected");
    backend.openBackend();
@@ -887,17 +884,12 @@
    assertTrue(backend.supports(BackendOperation.LDIF_EXPORT), "Export not supported");
    ByteArrayOutputStream ldifData = new ByteArrayOutputStream();
    LDIFExportConfig exportConfig = new LDIFExportConfig(ldifData);
    try (final LDIFExportConfig exportConfig = new LDIFExportConfig(ldifData))
    {
    exportConfig.setIncludeOperationalAttributes(true);
    exportConfig.setIncludeVirtualAttributes(true);
    try
    {
      backend.exportLDIF(exportConfig);
    }
    finally
    {
      exportConfig.close();
    }
    String ldifString = ldifData.toString();
    assertEquals(ldifString.contains(testBaseDN.toString()), true, "Export without rootDN");
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/StateTest.java
@@ -86,9 +86,9 @@
    when(serverContext.getDiskSpaceMonitor()).thenReturn(mock(DiskSpaceMonitor.class));
    storage = new PersistItStorage(createBackendCfg(), serverContext);
    org.opends.server.backends.pluggable.spi.Importer importer = storage.startImport();
    try(final org.opends.server.backends.pluggable.spi.Importer importer = storage.startImport()) {
    importer.createTree(stateTreeName);
    importer.close();
    }
    storage.open();
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/TestDnKeyFormat.java
@@ -361,9 +361,8 @@
    // Convert the test LDIF string to a byte array
    byte[] originalLDIFBytes = StaticUtils.getBytes(ldifString);
    LDIFReader reader = new LDIFReader(new LDIFImportConfig(
        new ByteArrayInputStream(originalLDIFBytes)));
    try (final LDIFReader reader = new LDIFReader(new LDIFImportConfig(new ByteArrayInputStream(originalLDIFBytes))))
    {
    Entry entryBefore, entryAfter;
    while ((entryBefore = reader.readEntry(false)) != null) {
      ByteString bytes = ID2Entry.entryToDatabase(entryBefore,
@@ -415,7 +414,7 @@
        }
      }
    }
    reader.close();
    }
  }
  /**
@@ -431,9 +430,8 @@
    // Convert the test LDIF string to a byte array
    byte[] originalLDIFBytes = StaticUtils.getBytes(ldifString);
    LDIFReader reader = new LDIFReader(new LDIFImportConfig(
        new ByteArrayInputStream(originalLDIFBytes)));
    try (final LDIFReader reader = new LDIFReader(new LDIFImportConfig(new ByteArrayInputStream(originalLDIFBytes))))
    {
    Entry entryBefore, entryAfterV1;
    while ((entryBefore = reader.readEntry(false)) != null) {
      ByteStringBuilder bsb = new ByteStringBuilder();
@@ -442,7 +440,7 @@
      assertEquals(entryBefore, entryAfterV1);
    }
    reader.close();
    }
  }
  /**
@@ -480,9 +478,8 @@
    // Convert the test LDIF string to a byte array
    byte[] originalLDIFBytes = StaticUtils.getBytes(ldifString);
    LDIFReader reader = new LDIFReader(new LDIFImportConfig(
        new ByteArrayInputStream(originalLDIFBytes)));
    try (final LDIFReader reader = new LDIFReader(new LDIFImportConfig(new ByteArrayInputStream(originalLDIFBytes))))
    {
    Entry entryBefore, entryAfterV2;
    while ((entryBefore = reader.readEntry(false)) != null) {
      ByteStringBuilder bsb = new ByteStringBuilder();
@@ -494,7 +491,7 @@
      }
      assertEquals(entryBefore, entryAfterV2);
    }
    reader.close();
    }
  }
  /**
@@ -511,9 +508,8 @@
    // Convert the test LDIF string to a byte array
    byte[] originalLDIFBytes = StaticUtils.getBytes(ldifString);
    LDIFReader reader = new LDIFReader(new LDIFImportConfig(
        new ByteArrayInputStream(originalLDIFBytes)));
    try (final LDIFReader reader = new LDIFReader(new LDIFImportConfig(new ByteArrayInputStream(originalLDIFBytes))))
    {
    Entry entryBefore, entryAfterV3;
    while ((entryBefore = reader.readEntry(false)) != null) {
      ByteStringBuilder bsb = new ByteStringBuilder();
@@ -525,7 +521,7 @@
      }
      assertEquals(entryBefore, entryAfterV3);
    }
    reader.close();
    }
  }
  @DataProvider
opendj-sdk/opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/Utils.java
@@ -51,8 +51,12 @@
    assertIdsEquals(actual.iterator(), expected);
  }
  public static EntryID id(long id) {
    return new EntryID(id);
  }
  private static List<EntryID> asList(long... array) {
    List<EntryID> list = new ArrayList<EntryID>(array.length);
    List<EntryID> list = new ArrayList<>(array.length);
    for(long l : array) {
      list.add(new EntryID(l));
    }