From d710c5f04c40abee7b9d79284f79a2295a96c214 Mon Sep 17 00:00:00 2001
From: Jean-Noël Rouvignac <jn.rouvignac@gmail.com>
Date: Mon, 31 Aug 2015 12:20:33 +0000
Subject: [PATCH] TasksTestCase.java: Extracted method waitTaskCompletedSuccessfully() from test classes.

---
 opendj-server-legacy/src/test/java/org/opends/server/tasks/DisconnectClientTaskTestCase.java |   37 ++++++++++---------------------------
 1 files changed, 10 insertions(+), 27 deletions(-)

diff --git a/opendj-server-legacy/src/test/java/org/opends/server/tasks/DisconnectClientTaskTestCase.java b/opendj-server-legacy/src/test/java/org/opends/server/tasks/DisconnectClientTaskTestCase.java
index 33ce461..5b6973e 100644
--- a/opendj-server-legacy/src/test/java/org/opends/server/tasks/DisconnectClientTaskTestCase.java
+++ b/opendj-server-legacy/src/test/java/org/opends/server/tasks/DisconnectClientTaskTestCase.java
@@ -26,32 +26,22 @@
  */
 package org.opends.server.tasks;
 
-
-
 import java.net.Socket;
 
-import org.testng.annotations.Test;
-import org.testng.annotations.BeforeClass;
-
-import org.opends.server.TestCaseUtils;
 import org.forgerock.i18n.LocalizableMessage;
-import org.opends.server.backends.task.Task;
-import org.opends.server.backends.task.TaskState;
+import org.forgerock.opendj.ldap.ByteString;
+import org.opends.server.TestCaseUtils;
 import org.opends.server.extensions.GetConnectionIDExtendedOperation;
 import org.opends.server.protocols.ldap.*;
 import org.opends.server.types.DN;
 import org.opends.server.util.StaticUtils;
-import org.forgerock.opendj.ldap.ByteString;
-
-import static org.testng.Assert.*;
+import org.testng.annotations.BeforeClass;
+import org.testng.annotations.Test;
 
 import static org.opends.server.util.ServerConstants.*;
+import static org.testng.Assert.*;
 
-
-
-/**
- * Tests the disconnect client task.
- */
+/** Tests the disconnect client task. */
 public class DisconnectClientTaskTestCase
        extends TasksTestCase
 {
@@ -127,13 +117,10 @@
       "ds-task-disconnect-notify-client: true",
       "ds-task-disconnect-message: " + disconnectMessage);
 
-    Task task = getCompletedTask(taskDN);
-    assertNotNull(task);
-    assertEquals(task.getTaskState(), TaskState.COMPLETED_SUCCESSFULLY);
+    waitTaskCompletedSuccessfully(taskDN);
 
 
-    // Make sure that we get a notice of disconnection on the initial
-    // connection.
+    // Make sure that we get a notice of disconnection on the initial connection.
     message = r.readMessage();
     extendedResponse = message.getExtendedResponseProtocolOp();
     assertEquals(extendedResponse.getOID(),
@@ -201,16 +188,12 @@
       "ds-task-disconnect-connection-id: " + connectionID,
       "ds-task-disconnect-notify-client: false");
 
-    Task task = getCompletedTask(taskDN);
-    assertNotNull(task);
-    assertEquals(task.getTaskState(), TaskState.COMPLETED_SUCCESSFULLY);
+    waitTaskCompletedSuccessfully(taskDN);
 
 
-    // Make sure that the client connection has been closed with no notice of
-    // disconnection.
+    // Make sure that the client connection has been closed with no notice of disconnection.
     assertNull(r.readMessage());
 
     StaticUtils.close(s);
   }
 }
-

--
Gitblit v1.10.0