Skip to content

Commit c327922

Browse files
committed
Add explanations to broken tests
1 parent 84380e4 commit c327922

5 files changed

Lines changed: 5 additions & 5 deletions

File tree

extractor/src/test/java/org/schabi/newpipe/extractor/services/peertube/PeertubeStreamExtractorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public void testGetLanguageInformation() throws ParsingException {
9696
@Override public int expectedStreamSegmentsCount() { return 0; }
9797
}
9898

99-
@Ignore("TODO fix")
99+
@Ignore("Test broken, SSL problem")
100100
public static class AgeRestricted extends DefaultStreamExtractorTest {
101101
private static final String ID = "dbd8e5e1-c527-49b6-b70c-89101dbb9c08";
102102
private static final String INSTANCE = "https://nocensoring.net";

extractor/src/test/java/org/schabi/newpipe/extractor/services/soundcloud/SoundcloudPlaylistExtractorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,7 @@ public void testRelatedItems() throws Exception {
362362
}
363363

364364
@Test
365-
@Ignore("TODO fix")
365+
@Ignore("Test broken? Playlist has 2 entries, each page has 1 entry meaning it has 2 pages.")
366366
public void testMoreRelatedItems() throws Exception {
367367
try {
368368
defaultTestMoreItems(extractor);

extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/YoutubePlaylistExtractorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public void nonExistentFetch() throws Exception {
5959
}
6060

6161
@Test(expected = ContentNotAvailableException.class)
62-
@Ignore("TODO fix")
62+
@Ignore("Broken, now invalid playlists redirect to youtube homepage")
6363
public void invalidId() throws Exception {
6464
final PlaylistExtractor extractor =
6565
YouTube.getPlaylistExtractor("https://www.youtube.com/playlist?list=INVALID_ID");

extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/stream/YoutubeStreamExtractorControversialTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
/**
2222
* Test for {@link YoutubeStreamLinkHandlerFactory}
2323
*/
24-
@Ignore("Video is not available anymore")
24+
@Ignore("Video is not available in specific countries. Someone else has to generate mocks")
2525
public class YoutubeStreamExtractorControversialTest extends DefaultStreamExtractorTest {
2626
private static final String RESOURCE_PATH = DownloaderFactory.RESOURCE_PATH + "services/youtube/extractor/stream/";
2727
private static final String ID = "T4XJQO3qol8";

extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/stream/YoutubeStreamExtractorDefaultTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ public void testDescription() throws Exception {
160160

161161
}
162162

163-
@Ignore("TODO fix")
163+
@Ignore("Test broken, video was made private")
164164
public static class RatingsDisabledTest extends DefaultStreamExtractorTest {
165165
private static final String ID = "HRKu0cvrr_o";
166166
private static final int TIMESTAMP = 17;

0 commit comments

Comments
 (0)