Removed SUCCESS as a terminal state

Azure · Oct 7, 2020 · 40e4e06 · 40e4e06
1 parent 68b4550
commit 40e4e06
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/...stration/src/main/java/com/azure/communication/administration/PhoneNumberAsyncClient.java b/...stration/src/main/java/com/azure/communication/administration/PhoneNumberAsyncClient.java
@@ -805,12 +805,11 @@ public PollerFlux<PhoneNumberSearch, PhoneNumberSearch> beginCreateSearch(
             getSearchById(pollingContext.getLatestResponse().getValue().getSearchId())
                 .flatMap(getSearchResponse -> {
                     SearchStatus status = getSearchResponse.getStatus();
-                    if (status.equals(SearchStatus.SUCCESS) 
-                        || status.equals(SearchStatus.EXPIRED) 
+                    if (status.equals(SearchStatus.EXPIRED) 
                         || status.equals(SearchStatus.CANCELLED) 
                         || status.equals(SearchStatus.RESERVED)) {
-                            return Mono.just(new PollResponse<>(
-                            LongRunningOperationStatus.SUCCESSFULLY_COMPLETED, getSearchResponse));
+                        return Mono.just(new PollResponse<>(
+                        LongRunningOperationStatus.SUCCESSFULLY_COMPLETED, getSearchResponse));
                     }
                     if (status.equals(SearchStatus.ERROR)) {
                         return Mono.just(new PollResponse<>(

diff --git a/...st/java/com/azure/communication/administration/PhoneNumberAsyncClientIntegrationTest.java b/...st/java/com/azure/communication/administration/PhoneNumberAsyncClientIntegrationTest.java
@@ -23,7 +23,6 @@
 import com.azure.communication.common.PhoneNumber;
 import com.azure.core.http.rest.PagedFlux;
 import com.azure.core.http.rest.Response;
-import com.azure.core.util.polling.PollerFlux;
 import com.azure.core.util.polling.SyncPoller;
 import com.azure.core.util.Context;
 import org.junit.jupiter.api.Test;