[feed] Try to make tests more stable (#9315)

Signed-off-by: Fabian Wolter <github@fabian-wolter.de>
This commit is contained in:
Fabian Wolter 2020-12-12 19:30:28 +01:00 committed by GitHub
parent 70adea298a
commit 2bcb548381
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -104,6 +104,7 @@ public class FeedHandlerTest extends JavaOSGiTest {
private Thing feedThing; private Thing feedThing;
private FeedHandler feedHandler; private FeedHandler feedHandler;
private ChannelUID channelUID; private ChannelUID channelUID;
private HttpService httpService;
/** /**
* This class is used as a mock for HTTP web server, serving XML feed content. * This class is used as a mock for HTTP web server, serving XML feed content.
@ -175,9 +176,8 @@ public class FeedHandlerTest extends JavaOSGiTest {
}); });
} }
private void registerFeedTestServlet() { private synchronized void registerFeedTestServlet() {
HttpService httpService = getService(HttpService.class); waitForAssert(() -> assertThat(httpService = getService(HttpService.class), is(notNullValue())));
assertThat(httpService, is(notNullValue()));
servlet = new FeedServiceMock(DEFAULT_MOCK_CONTENT); servlet = new FeedServiceMock(DEFAULT_MOCK_CONTENT);
try { try {
httpService.registerServlet(MOCK_SERVLET_PATH, servlet, null, null); httpService.registerServlet(MOCK_SERVLET_PATH, servlet, null, null);
@ -186,9 +186,8 @@ public class FeedHandlerTest extends JavaOSGiTest {
} }
} }
private void unregisterFeedTestServlet() { private synchronized void unregisterFeedTestServlet() {
HttpService httpService = getService(HttpService.class); waitForAssert(() -> assertThat(httpService = getService(HttpService.class), is(notNullValue())));
assertThat(httpService, is(notNullValue()));
httpService.unregister(MOCK_SERVLET_PATH); httpService.unregister(MOCK_SERVLET_PATH);
servlet = null; servlet = null;
} }