From 69c227a9f1d91268fbd91471b012af38fba8b857 Mon Sep 17 00:00:00 2001 From: liamfallon Date: Mon, 6 Apr 2020 15:41:28 +0100 Subject: Fix intermittent unit test failures WebSocket The Web Socket server is marked as started before the start process has complted. This change fixes that issue. Issue-ID: POLICY-1470 Change-Id: I5b5ef5832893d884a6551b0516df91cb71d6c622 Signed-off-by: liamfallon --- .../service/engine/engdep/EngDepMessagingServiceTest.java | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to 'services/services-engine/src') diff --git a/services/services-engine/src/test/java/org/onap/policy/apex/service/engine/engdep/EngDepMessagingServiceTest.java b/services/services-engine/src/test/java/org/onap/policy/apex/service/engine/engdep/EngDepMessagingServiceTest.java index 901cbeac4..42a3a55d8 100644 --- a/services/services-engine/src/test/java/org/onap/policy/apex/service/engine/engdep/EngDepMessagingServiceTest.java +++ b/services/services-engine/src/test/java/org/onap/policy/apex/service/engine/engdep/EngDepMessagingServiceTest.java @@ -1,27 +1,30 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2020 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at - * + * * http://www.apache.org/licenses/LICENSE-2.0 - * + * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. - * + * * SPDX-License-Identifier: Apache-2.0 * ============LICENSE_END========================================================= */ package org.onap.policy.apex.service.engine.engdep; +import static org.awaitility.Awaitility.await; import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; + +import java.util.concurrent.TimeUnit; import org.junit.Before; import org.junit.Test; @@ -56,10 +59,10 @@ public class EngDepMessagingServiceTest { @Test public void testStartStop() throws ApexException { edMessagingService.start(); - assertTrue(edMessagingService.isStarted()); + await().atMost(1, TimeUnit.SECONDS).until(() -> edMessagingService.isStarted()); assertFalse(edMessagingService.isStopped()); edMessagingService.stop(); - assertTrue(edMessagingService.isStopped()); + await().atMost(1, TimeUnit.SECONDS).until(() -> edMessagingService.isStopped()); assertFalse(edMessagingService.isStarted()); } } -- cgit 1.2.3-korg