ADAP-207: fixed few tests

wenjunche · wenjunche · commit 2bfa54c4bd93 · 2022-02-19T22:00:10.000-05:00
diff --git a/src/test/java/com/openfin/desktop/ChannelTest.java b/src/test/java/com/openfin/desktop/ChannelTest.java
@@ -307,7 +307,7 @@ public JSONObject invoke(String action, Object payload, JSONObject senderIdentit
 	@Test
 	public void connectionListener() throws Exception {
 		final String channelName = "connectionListenerTest";
-		CountDownLatch latch = new CountDownLatch(2);
+		CountDownLatch latch = new CountDownLatch(3);
 		desktopConnection.getChannel(channelName).createAsync().thenAccept(provider -> {
 			provider.addProviderListener(new ChannelProviderListener() {
 				@Override
@@ -321,6 +321,15 @@ public void onClientDisconnect(ChannelClientConnectEvent connectionEvent) {
 			});
 
 			desktopConnection.getChannel(channelName).connectAsync().thenAccept(client -> {
+				client.addChannelListener(new ChannelListener() {
+					@Override
+					public void onChannelConnect(ConnectionEvent connectionEvent) {
+					}
+					@Override
+					public void onChannelDisconnect(ConnectionEvent connectionEvent) {
+						latch.countDown();
+					}
+				});
 				client.disconnect(null);
 			});
 		});
diff --git a/src/test/java/com/openfin/desktop/InteropTest.java b/src/test/java/com/openfin/desktop/InteropTest.java
@@ -35,6 +35,7 @@ public static void setup() throws Exception {
 		cfg.setManifestLocation("https://testing-assets.openfin.co/adapters/interop/app.json");
 //		cfg.setManifestLocation("http://localhost:5555/app.json");
 		desktopConnection = TestUtils.setupConnection(DESKTOP_UUID, cfg);
+		Thread.sleep(3000);  // give time for the js app to start up and configure everything properly
 	}
 
 	@AfterClass

Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@ public static void setup() throws Exception {`
`35`	`35`	`cfg.setManifestLocation("https://testing-assets.openfin.co/adapters/interop/app.json");`
`36`	`36`	`// cfg.setManifestLocation("http://localhost:5555/app.json");`
`37`	`37`	`desktopConnection = TestUtils.setupConnection(DESKTOP_UUID, cfg);`
	`38`	`+ Thread.sleep(3000); // give time for the js app to start up and configure everything properly`
`38`	`39`	`}`
`39`	`40`
`40`	`41`	`@AfterClass`