Skip to content

Commit

Permalink
fix: Drops youtube sharing video tests.
Browse files Browse the repository at this point in the history
We started seeing errors in the tests. Seems to be only from certain IPs running the tests regularly.

 [modules/UI/videolayout/VideoContainer.js] <HTMLVideoElement.<anonymous>>:  abort handler was called for the large video.
 [react/features/shared-video/components/web/AbstractVideoManager.ts] <P7.onError>:  Error in the video player 150 Check error code at https://developers.google.com/youtube/iframe_api_reference#onError
  • Loading branch information
damencho committed Aug 9, 2024
1 parent f8dcab2 commit 759723c
Showing 1 changed file with 2 additions and 19 deletions.
21 changes: 2 additions & 19 deletions src/test/java/org/jitsi/meet/test/SharedVideoTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,9 @@ public class SharedVideoTest
extends WebTestBase
{
/**
* Links to youtube videos and their expected urls, used for loading and
* Links to videos and their expected urls, used for loading and
* testing.
*/
private static String V1_VIDEO_ID = "xNXN7CZk8X0";
private static String V1_LINK = "https://www.youtube.com/watch?v=" + V1_VIDEO_ID;
public static String V2_LINK = "https://archive.org/download/BigBuckBunny_124/Content/big_buck_bunny_720p_surround.mp4";
private static String V2_VIDEO_ID = V2_LINK;

Expand Down Expand Up @@ -76,7 +74,7 @@ public void setupClass()
@Test
public void startSharingVideo()
{
startSharingVideoByUrl(V1_LINK, V1_VIDEO_ID, true);
startSharingVideoByUrl(V2_LINK, V2_VIDEO_ID, true);

checkRunning();

Expand Down Expand Up @@ -376,22 +374,7 @@ public void stopSharingTest()
}
}

/**
* Starts sharing another video.
*/
@Test(dependsOnMethods = { "stopSharingTest" })
public void startSharingDifferentVideo()
{
startSharingVideoByUrl(V2_LINK, V2_VIDEO_ID, true);

checkRunning();

checkUrls();

stopSharingTest();
}

@Test(dependsOnMethods = { "startSharingDifferentVideo" })
public void startSharingDifferentVideoAfterStop()
{
startSharingVideoByUrl(V2_LINK, V2_VIDEO_ID, true);
Expand Down

0 comments on commit 759723c

Please sign in to comment.