diff --git a/src/test/java/org/jitsi/meet/test/IFrameAPICommandsTest.java b/src/test/java/org/jitsi/meet/test/IFrameAPICommandsTest.java index ea5032298..7fe75ae1f 100644 --- a/src/test/java/org/jitsi/meet/test/IFrameAPICommandsTest.java +++ b/src/test/java/org/jitsi/meet/test/IFrameAPICommandsTest.java @@ -214,63 +214,65 @@ public void testCommandToggleLobby() @Test(dependsOnMethods = {"testCommandToggleLobby"}) public void testCommandStartStopShareVideo() { + // Currently most of the code of the test is commented temporary because of restrictions on the URLs we test + // with. Once a proper URL whitelist for shared video is implemented we should uncomment the code. hangUpAllParticipants(); - this.iFrameUrl = getIFrameUrl(null, null); - ensureTwoParticipants(this.iFrameUrl, null); - - WebParticipant participant1 = getParticipant1(); - WebParticipant participant2 = getParticipant2(); - WebDriver driver1 = participant1.getDriver(); - WebDriver driver2 = participant2.getDriver(); - - switchToIframeAPI(driver1); - - TestUtils.executeScript(driver1, - "window.jitsiAPI.executeCommand('startShareVideo', '" + SharedVideoTest.V2_LINK + "');"); - - switchToMeetContent(this.iFrameUrl, driver1); - - TestUtils.waitForDisplayedElementByID( - driver1, - "sharedVideoPlayer", - 10); - - // Now let's check the second participant state - // make sure we are in meet, not in the frame - driver2.switchTo().defaultContent(); - TestUtils.waitForDisplayedElementByID( - driver2, - "sharedVideoPlayer", - 10); - - switchToIframeAPI(driver1); - - TestUtils.executeScript(driver1, - "window.jitsiAPI.executeCommand('stopShareVideo');"); - - switchToMeetContent(this.iFrameUrl, driver1); - - try - { - TestUtils.waitForNotDisplayedElementByID( - driver1, "sharedVideoPlayer", 5); - } - catch(StaleElementReferenceException ex) - { - // if the element is detached in a process of checking its display - // status, means it is not visible anymore - } - try - { - TestUtils.waitForNotDisplayedElementByID( - driver2, "sharedVideoPlayer", 5); - } - catch(StaleElementReferenceException ex) - { - // if the element is detached in a process of checking its display - // status, means it is not visible anymore - } + // this.iFrameUrl = getIFrameUrl(null, null); + // ensureTwoParticipants(this.iFrameUrl, null); + + // WebParticipant participant1 = getParticipant1(); + // WebParticipant participant2 = getParticipant2(); + // WebDriver driver1 = participant1.getDriver(); + // WebDriver driver2 = participant2.getDriver(); + + // switchToIframeAPI(driver1); + + // TestUtils.executeScript(driver1, + // "window.jitsiAPI.executeCommand('startShareVideo', '" + SharedVideoTest.V2_LINK + "');"); + + // switchToMeetContent(this.iFrameUrl, driver1); + + // TestUtils.waitForDisplayedElementByID( + // driver1, + // "sharedVideoPlayer", + // 10); + + // // Now let's check the second participant state + // // make sure we are in meet, not in the frame + // driver2.switchTo().defaultContent(); + // TestUtils.waitForDisplayedElementByID( + // driver2, + // "sharedVideoPlayer", + // 10); + + // switchToIframeAPI(driver1); + + // TestUtils.executeScript(driver1, + // "window.jitsiAPI.executeCommand('stopShareVideo');"); + + // switchToMeetContent(this.iFrameUrl, driver1); + + // try + // { + // TestUtils.waitForNotDisplayedElementByID( + // driver1, "sharedVideoPlayer", 5); + // } + // catch(StaleElementReferenceException ex) + // { + // // if the element is detached in a process of checking its display + // // status, means it is not visible anymore + // } + // try + // { + // TestUtils.waitForNotDisplayedElementByID( + // driver2, "sharedVideoPlayer", 5); + // } + // catch(StaleElementReferenceException ex) + // { + // // if the element is detached in a process of checking its display + // // status, means it is not visible anymore + // } } /** diff --git a/src/test/resources/desktop/testng.xml b/src/test/resources/desktop/testng.xml index 3cf56006d..6a2b585bd 100644 --- a/src/test/resources/desktop/testng.xml +++ b/src/test/resources/desktop/testng.xml @@ -198,11 +198,11 @@ - +