Skip to content

Commit a13d4a8

Browse files
committed
Fixed more tests
1 parent c6d9063 commit a13d4a8

File tree

7 files changed

+17
-5
lines changed

7 files changed

+17
-5
lines changed

extractor/src/main/java/org/schabi/newpipe/extractor/services/peertube/extractors/PeertubeStreamExtractor.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,6 @@ public class PeertubeStreamExtractor extends StreamExtractor {
6565
private static final String FILE_DOWNLOAD_URL = "fileDownloadUrl";
6666
private static final String FILE_URL = "fileUrl";
6767
private static final String PLAYLIST_URL = "playlistUrl";
68-
private static final String RESOLUTION_ID = "resolution.id";
6968
private static final String STREAMING_PLAYLISTS = "streamingPlaylists";
7069

7170
private final String baseUrl;

extractor/src/test/java/org/schabi/newpipe/extractor/services/bandcamp/BandcampRadioStreamExtractorTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ public void testGettingCorrectStreamExtractor() throws ExtractionException {
4949
@Override public String expectedId() throws Exception { return "230"; }
5050
@Override public String expectedUrlContains() throws Exception { return URL; }
5151
@Override public String expectedOriginalUrlContains() throws Exception { return URL; }
52+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
5253
@Override public boolean expectedHasVideoAndAudioStreams() { return false; }
5354
@Override public boolean expectedHasSubtitles() { return false; }
5455
@Override public boolean expectedHasFrames() { return false; }

extractor/src/test/java/org/schabi/newpipe/extractor/services/bandcamp/BandcampStreamExtractorTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,11 @@ public long expectedDislikeCountAtLeast() {
118118
return Long.MIN_VALUE;
119119
}
120120

121+
@Override
122+
public boolean expectedHasVideoOnlyStreams() {
123+
return false;
124+
}
125+
121126
@Override
122127
public boolean expectedHasVideoAndAudioStreams() {
123128
return false;

extractor/src/test/java/org/schabi/newpipe/extractor/services/media_ccc/MediaCCCStreamExtractorTest.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ public static void setUp() throws Exception {
5252
@Override public long expectedLikeCountAtLeast() { return -1; }
5353
@Override public long expectedDislikeCountAtLeast() { return -1; }
5454
@Override public boolean expectedHasRelatedItems() { return false; }
55+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
5556
@Override public boolean expectedHasSubtitles() { return false; }
5657
@Override public boolean expectedHasFrames() { return false; }
5758
@Override public List<String> expectedTags() { return Arrays.asList("gpn18", "105"); }
@@ -74,8 +75,8 @@ public void testUploaderAvatarUrl() throws Exception {
7475

7576
@Override
7677
@Test
77-
public void testVideoOnlyStreams() throws Exception {
78-
super.testVideoOnlyStreams();
78+
public void testVideoAudioStreams() throws Exception {
79+
super.testVideoAudioStreams();
7980
assertEquals(4, extractor.getVideoStreams().size());
8081
}
8182

@@ -123,6 +124,7 @@ public static void setUp() throws Exception {
123124
@Override public long expectedLikeCountAtLeast() { return -1; }
124125
@Override public long expectedDislikeCountAtLeast() { return -1; }
125126
@Override public boolean expectedHasRelatedItems() { return false; }
127+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
126128
@Override public boolean expectedHasSubtitles() { return false; }
127129
@Override public boolean expectedHasFrames() { return false; }
128130
@Override public List<String> expectedTags() { return Arrays.asList("36c3", "10565", "2019", "Security", "Main"); }
@@ -143,8 +145,8 @@ public void testUploaderAvatarUrl() throws Exception {
143145

144146
@Override
145147
@Test
146-
public void testVideoOnlyStreams() throws Exception {
147-
super.testVideoOnlyStreams();
148+
public void testVideoAudioStreams() throws Exception {
149+
super.testVideoAudioStreams();
148150
assertEquals(8, extractor.getVideoStreams().size());
149151
}
150152

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
public abstract class PeertubeStreamExtractorTest extends DefaultStreamExtractorTest {
2525
private static final String BASE_URL = "/videos/watch/";
2626

27+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
2728
@Override public boolean expectedHasAudioStreams() { return false; }
2829
@Override public boolean expectedHasFrames() { return false; }
2930

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ public static void setUp() throws Exception {
6767
@Override public long expectedLikeCountAtLeast() { return -1; }
6868
@Override public long expectedDislikeCountAtLeast() { return -1; }
6969
@Override public boolean expectedHasAudioStreams() { return false; }
70+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
7071
@Override public boolean expectedHasVideoAndAudioStreams() { return false; }
7172
@Override public boolean expectedHasSubtitles() { return false; }
7273
@Override public boolean expectedHasFrames() { return false; }
@@ -130,6 +131,7 @@ public void testRelatedItems() throws Exception {
130131
@Override public long expectedLikeCountAtLeast() { return -1; }
131132
@Override public long expectedDislikeCountAtLeast() { return -1; }
132133
@Override public boolean expectedHasAudioStreams() { return false; }
134+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
133135
@Override public boolean expectedHasVideoAndAudioStreams() { return false; }
134136
@Override public boolean expectedHasRelatedItems() { return true; }
135137
@Override public boolean expectedHasSubtitles() { return false; }
@@ -172,6 +174,7 @@ public static void setUp() throws Exception {
172174
@Nullable @Override public String expectedTextualUploadDate() { return "2019-03-28 13:36:18"; }
173175
@Override public long expectedLikeCountAtLeast() { return -1; }
174176
@Override public long expectedDislikeCountAtLeast() { return -1; }
177+
@Override public boolean expectedHasVideoOnlyStreams() { return false; }
175178
@Override public boolean expectedHasVideoAndAudioStreams() { return false; }
176179
@Override public boolean expectedHasSubtitles() { return false; }
177180
@Override public boolean expectedHasFrames() { return false; }

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ public void testUploaderName() throws Exception {
6060
@Nullable @Override public String expectedTextualUploadDate() { return "2020-02-22"; }
6161
@Override public long expectedLikeCountAtLeast() { return 825000; }
6262
@Override public long expectedDislikeCountAtLeast() { return -1; }
63+
@Override public boolean expectedHasVideoAndAudioStreams() { return false; }
6364
@Override public boolean expectedHasSubtitles() { return false; }
6465
@Nullable @Override public String expectedDashMpdUrlContains() { return "https://manifest.googlevideo.com/api/manifest/dash/"; }
6566
@Override public boolean expectedHasFrames() { return false; }

0 commit comments

Comments
 (0)