ADAP-83: fixed noficationTest for Runtime 6.0

wenjunche · wenjunche · commit 4bbd77262a23 · 2017-07-10T13:06:40.000-04:00
diff --git a/src/test/java/com/openfin/desktop/NotificationTest.java b/src/test/java/com/openfin/desktop/NotificationTest.java
@@ -32,6 +32,10 @@ public static void setup() throws Exception {
         logger.debug("starting");
         desktopConnection = TestUtils.setupConnection(DESKTOP_UUID);
 
+        // for Runtime 6.0+, needs to start at least one app for notifications to work
+        ApplicationOptions options = TestUtils.getAppOptions(null);
+        Application application = TestUtils.runApplication(options, desktopConnection);
+
         logger.info("Waiting for notification center to be ready");
         // @TODO currently there is no way to know Notification center is ready, so we just sleep here
         // @TODO we will fix notification center to generate an event when it finishes initialization and is ready to process requests
diff --git a/src/test/java/com/openfin/desktop/TestUtils.java b/src/test/java/com/openfin/desktop/TestUtils.java
@@ -109,11 +109,13 @@ public static DesktopConnection setupConnection(String connectionUuid, String rd
         }
 
         DesktopConnection desktopConnection = new DesktopConnection(connectionUuid);
-        desktopConnection.setAdditionalRuntimeArguments(" --v=1 --no-sandbox ");  // turn on Chromium debug log
-        desktopConnection.setDevToolsPort(9090);
-        desktopConnection.setRdmUrl(rdmUrl);
-        desktopConnection.setRuntimeAssetsUrl(assetsUrl);
-        desktopConnection.connectToVersion(runtimeVersion, new DesktopStateListener() {
+        RuntimeConfiguration configuration = new RuntimeConfiguration();
+        configuration.setRuntimeVersion(runtimeVersion);
+        configuration.setAdditionalRuntimeArguments(" --v=1 --no-sandbox ");  // turn on Chromium debug log
+        configuration.setDevToolsPort(9090);
+        configuration.setRdmURL(rdmUrl);
+        configuration.setRuntimeAssetURL(assetsUrl);
+        desktopConnection.connect(configuration, new DesktopStateListener() {
             @Override
             public void onReady() {
                 logger.info("Connected to OpenFin runtime");
