Added assertion before cast, and renamed to make what (is now called) getPreviousResponseAs(...) is doing more clear.

Author: TinaTiel

src/integrationTest/java/net/twasi/obsremotejava/test/AbstractObsE2ETest.java

@@ -1,5 +1,6 @@
 package net.twasi.obsremotejava.test;
 
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.fail;
 
 import java.util.concurrent.BlockingQueue;
@@ -124,8 +125,10 @@ protected void waitReasonably(long ms) {
     resultQueue.add(obj);
   };
 
-  protected <T> T getResponseAs(Class<T> clazz) {
-    return clazz.cast(resultQueue.remove());
+  protected <T> T getPreviousResponseAs(Class<T> clazz) {
+    Object previousResponse = resultQueue.remove();
+    assertThat(previousResponse).isInstanceOf(clazz);
+    return clazz.cast(previousResponse);
   }
 
 }

src/integrationTest/java/net/twasi/obsremotejava/test/ObsRemoteE2eIT.java

@@ -61,7 +61,7 @@ void getScenes() {
     waitReasonably();
 
     // Then scenes match as expected
-    GetSceneListResponse res = getResponseAs(GetSceneListResponse.class);
+    GetSceneListResponse res = getPreviousResponseAs(GetSceneListResponse.class);
     assertThat(res.getScenes().stream().map(Scene::getName).collect(Collectors.toList()))
       .usingRecursiveComparison().ignoringCollectionOrder().isEqualTo(scenes);
 
@@ -103,7 +103,7 @@ void getSourcesList() {
     waitReasonably();
 
     // Then it matches as expected
-    GetSourcesListResponse res = getResponseAs(GetSourcesListResponse.class);
+    GetSourcesListResponse res = getPreviousResponseAs(GetSourcesListResponse.class);
     List<Source> sources = res.getSources();
     List<String> actualNames = sources.stream().map(Source::getName).collect(Collectors.toList());
     assertThat(actualNames.size()).isEqualTo(sources.size());