Skip to content

Commit 92b199d

Browse files
evermind-zzInfinityLoop1308
authored andcommitted
searchfilters: make tests compile -- they are not verified if they are actual working
1 parent 45c9c30 commit 92b199d

8 files changed

Lines changed: 53 additions & 53 deletions

File tree

extractor/src/test/java/org/schabi/newpipe/extractor/services/media_ccc/search/MediaCCCSearchExtractorTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212

1313
import static java.util.Collections.singletonList;
1414
import static org.schabi.newpipe.extractor.ServiceList.MediaCCC;
15-
import static org.schabi.newpipe.extractor.services.media_ccc.linkHandler.MediaCCCSearchQueryHandlerFactory.CONFERENCES;
16-
import static org.schabi.newpipe.extractor.services.media_ccc.linkHandler.MediaCCCSearchQueryHandlerFactory.EVENTS;
15+
import static com.example.newpipesearchfilters.filter.MediaCCCFilters.CONFERENCES;
16+
import static com.example.newpipesearchfilters.filter.MediaCCCFilters.EVENTS;
1717

1818
public class MediaCCCSearchExtractorTest {
1919
public static class All extends DefaultSearchExtractorTest {
@@ -46,7 +46,7 @@ public static class Conferences extends DefaultSearchExtractorTest {
4646
@BeforeAll
4747
public static void setUp() throws Exception {
4848
NewPipe.init(DownloaderTestImpl.getInstance());
49-
extractor = MediaCCC.getSearchExtractor(QUERY, singletonList(CONFERENCES), "");
49+
extractor = MediaCCC.getSearchExtractor(QUERY, singletonList(CONFERENCES), null);
5050
extractor.fetchPage();
5151
}
5252

@@ -70,7 +70,7 @@ public static class Events extends DefaultSearchExtractorTest {
7070
@BeforeAll
7171
public static void setUp() throws Exception {
7272
NewPipe.init(DownloaderTestImpl.getInstance());
73-
extractor = MediaCCC.getSearchExtractor(QUERY, singletonList(EVENTS), "");
73+
extractor = MediaCCC.getSearchExtractor(QUERY, singletonList(EVENTS), null);
7474
extractor.fetchPage();
7575
}
7676

extractor/src/test/java/org/schabi/newpipe/extractor/services/peertube/search/PeertubeSearchExtractorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public static void setUp() throws Exception {
5454
NewPipe.init(DownloaderTestImpl.getInstance());
5555
// setting instance might break test when running in parallel
5656
PeerTube.setInstance(new PeertubeInstance("https://framatube.org", "Framatube"));
57-
extractor = PeerTube.getSearchExtractor(QUERY, singletonList(PeertubeSearchQueryHandlerFactory.SEPIA_VIDEOS), "");
57+
extractor = PeerTube.getSearchExtractor(QUERY, singletonList(PeertubeSearchQueryHandlerFactory.SEPIA_VIDEOS), null);
5858
extractor.fetchPage();
5959
}
6060

@@ -73,7 +73,7 @@ public static class PagingTest {
7373
@Disabled("Exception in CI: javax.net.ssl.SSLHandshakeException: PKIX path validation failed: java.security.cert.CertPathValidatorException: validity check failed")
7474
public void duplicatedItemsCheck() throws Exception {
7575
NewPipe.init(DownloaderTestImpl.getInstance());
76-
final SearchExtractor extractor = PeerTube.getSearchExtractor("internet", singletonList(VIDEOS), "");
76+
final SearchExtractor extractor = PeerTube.getSearchExtractor("internet", singletonList(VIDEOS), null);
7777
extractor.fetchPage();
7878

7979
final InfoItemsPage<InfoItem> page1 = extractor.getInitialPage();

extractor/src/test/java/org/schabi/newpipe/extractor/services/peertube/search/PeertubeSearchQHTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public void testRegularValues() throws Exception {
2424
assertEquals("https://peertube.mastodon.host/api/v1/search/videos?search=Poifj%26jaijf", PeerTube.getSearchQHFactory().fromQuery("Poifj&jaijf").getUrl());
2525
assertEquals("https://peertube.mastodon.host/api/v1/search/videos?search=G%C3%BCl%C3%BCm", PeerTube.getSearchQHFactory().fromQuery("Gülüm").getUrl());
2626
assertEquals("https://peertube.mastodon.host/api/v1/search/videos?search=%3Fj%24%29H%C2%A7B", PeerTube.getSearchQHFactory().fromQuery("?j$)H§B").getUrl());
27-
assertEquals("https://sepiasearch.org/api/v1/search/videos?search=%3Fj%24%29H%C2%A7B", PeerTube.getSearchQHFactory().fromQuery("?j$)H§B", singletonList(PeertubeSearchQueryHandlerFactory.SEPIA_VIDEOS), "").getUrl());
28-
assertEquals("https://anotherpeertubeindex.com/api/v1/search/videos?search=%3Fj%24%29H%C2%A7B", PeerTube.getSearchQHFactory().fromQuery("?j$)H§B", singletonList(PeertubeSearchQueryHandlerFactory.SEPIA_VIDEOS), "", "https://anotherpeertubeindex.com").getUrl());
27+
assertEquals("https://sepiasearch.org/api/v1/search/videos?search=%3Fj%24%29H%C2%A7B", PeerTube.getSearchQHFactory().fromQuery("?j$)H§B", singletonList(PeertubeSearchQueryHandlerFactory.SEPIA_VIDEOS), null).getUrl());
28+
assertEquals("https://anotherpeertubeindex.com/api/v1/search/videos?search=%3Fj%24%29H%C2%A7B", PeerTube.getSearchQHFactory().fromQuery("?j$)H§B", singletonList(PeertubeSearchQueryHandlerFactory.SEPIA_VIDEOS), null, "https://anotherpeertubeindex.com").getUrl());
2929
}
3030
}

extractor/src/test/java/org/schabi/newpipe/extractor/services/soundcloud/search/SoundcloudSearchExtractorTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public static class Tracks extends DefaultSearchExtractorTest {
5757
@BeforeAll
5858
public static void setUp() throws Exception {
5959
NewPipe.init(DownloaderTestImpl.getInstance());
60-
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(TRACKS), "");
60+
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(TRACKS), null);
6161
extractor.fetchPage();
6262
}
6363

@@ -81,7 +81,7 @@ public static class Users extends DefaultSearchExtractorTest {
8181
@BeforeAll
8282
public static void setUp() throws Exception {
8383
NewPipe.init(DownloaderTestImpl.getInstance());
84-
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(USERS), "");
84+
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(USERS), null);
8585
extractor.fetchPage();
8686
}
8787

@@ -105,7 +105,7 @@ public static class Playlists extends DefaultSearchExtractorTest {
105105
@BeforeAll
106106
public static void setUp() throws Exception {
107107
NewPipe.init(DownloaderTestImpl.getInstance());
108-
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(PLAYLISTS), "");
108+
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(PLAYLISTS), null);
109109
extractor.fetchPage();
110110
}
111111

@@ -126,7 +126,7 @@ public static class PagingTest {
126126
@Test
127127
public void duplicatedItemsCheck() throws Exception {
128128
NewPipe.init(DownloaderTestImpl.getInstance());
129-
final SearchExtractor extractor = SoundCloud.getSearchExtractor("cirque du soleil", singletonList(TRACKS), "");
129+
final SearchExtractor extractor = SoundCloud.getSearchExtractor("cirque du soleil", singletonList(TRACKS), null);
130130
extractor.fetchPage();
131131

132132
final InfoItemsPage<InfoItem> page1 = extractor.getInitialPage();
@@ -151,7 +151,7 @@ public static class UserVerified extends DefaultSearchExtractorTest {
151151
@BeforeAll
152152
public static void setUp() throws Exception {
153153
NewPipe.init(DownloaderTestImpl.getInstance());
154-
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(USERS), "");
154+
extractor = SoundCloud.getSearchExtractor(QUERY, singletonList(USERS), null);
155155
extractor.fetchPage();
156156
}
157157

extractor/src/test/java/org/schabi/newpipe/extractor/services/soundcloud/search/SoundcloudSearchQHTest.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,21 +39,21 @@ public void testRegularValues() throws Exception {
3939
@Test
4040
public void testGetContentFilter() throws Exception {
4141
assertEquals("tracks", SoundCloud.getSearchQHFactory()
42-
.fromQuery("", asList(new String[]{"tracks"}), "").getContentFilters().get(0));
42+
.fromQuery("", asList(new String[]{"tracks"}), null).getContentFilters().get(0));
4343
assertEquals("users", SoundCloud.getSearchQHFactory()
44-
.fromQuery("asdf", asList(new String[]{"users"}), "").getContentFilters().get(0));
44+
.fromQuery("asdf", asList(new String[]{"users"}), null).getContentFilters().get(0));
4545
}
4646

4747
@Test
4848
public void testWithContentfilter() throws Exception {
4949
assertEquals("https://api-v2.soundcloud.com/search/tracks?q=asdf&limit=10&offset=0", removeClientId(SoundCloud.getSearchQHFactory()
50-
.fromQuery("asdf", asList(new String[]{TRACKS}), "").getUrl()));
50+
.fromQuery("asdf", asList(new String[]{TRACKS}), null).getUrl()));
5151
assertEquals("https://api-v2.soundcloud.com/search/users?q=asdf&limit=10&offset=0", removeClientId(SoundCloud.getSearchQHFactory()
52-
.fromQuery("asdf", asList(new String[]{USERS}), "").getUrl()));
52+
.fromQuery("asdf", asList(new String[]{USERS}), null).getUrl()));
5353
assertEquals("https://api-v2.soundcloud.com/search/playlists?q=asdf&limit=10&offset=0", removeClientId(SoundCloud.getSearchQHFactory()
54-
.fromQuery("asdf", asList(new String[]{PLAYLISTS}), "").getUrl()));
54+
.fromQuery("asdf", asList(new String[]{PLAYLISTS}), null).getUrl()));
5555
assertEquals("https://api-v2.soundcloud.com/search?q=asdf&limit=10&offset=0", removeClientId(SoundCloud.getSearchQHFactory()
56-
.fromQuery("asdf", asList(new String[]{"fjiijie"}), "").getUrl()));
56+
.fromQuery("asdf", asList(new String[]{"fjiijie"}), null).getUrl()));
5757
}
5858

5959
@Test
@@ -68,7 +68,7 @@ public void testGetAvailableContentFilter() {
6868

6969
@Test
7070
public void testGetAvailableSortFilter() {
71-
final String[] contentFilter = SoundCloud.getSearchQHFactory().getAvailableSortFilter();
72-
assertEquals(0, contentFilter.length);
71+
// TODO evermind-zz final String[] contentFilter = SoundCloud.getSearchQHFactory().getAvailableSortFilter();
72+
// TODO evermind-zz assertEquals(0, contentFilter.length);
7373
}
7474
}

extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/search/YoutubeMusicSearchExtractorTest.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public static class MusicSongs extends DefaultSearchExtractorTest {
2727
@BeforeAll
2828
public static void setUp() throws Exception {
2929
NewPipe.init(DownloaderTestImpl.getInstance());
30-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_SONGS), "");
30+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_SONGS), null);
3131
extractor.fetchPage();
3232
}
3333

@@ -49,7 +49,7 @@ public static class MusicVideos extends DefaultSearchExtractorTest {
4949
@BeforeAll
5050
public static void setUp() throws Exception {
5151
NewPipe.init(DownloaderTestImpl.getInstance());
52-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_VIDEOS), "");
52+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_VIDEOS), null);
5353
extractor.fetchPage();
5454
}
5555

@@ -71,7 +71,7 @@ public static class MusicAlbums extends DefaultSearchExtractorTest {
7171
@BeforeAll
7272
public static void setUp() throws Exception {
7373
NewPipe.init(DownloaderTestImpl.getInstance());
74-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_ALBUMS), "");
74+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_ALBUMS), null);
7575
extractor.fetchPage();
7676
}
7777

@@ -93,7 +93,7 @@ public static class MusicPlaylists extends DefaultSearchExtractorTest {
9393
@BeforeAll
9494
public static void setUp() throws Exception {
9595
NewPipe.init(DownloaderTestImpl.getInstance());
96-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_PLAYLISTS), "");
96+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_PLAYLISTS), null);
9797
extractor.fetchPage();
9898
}
9999

@@ -116,7 +116,7 @@ public static class MusicArtists extends DefaultSearchExtractorTest {
116116
@BeforeAll
117117
public static void setUp() throws Exception {
118118
NewPipe.init(DownloaderTestImpl.getInstance());
119-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_ARTISTS), "");
119+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_ARTISTS), null);
120120
extractor.fetchPage();
121121
}
122122

@@ -140,7 +140,7 @@ public static class Suggestion extends DefaultSearchExtractorTest {
140140
@BeforeAll
141141
public static void setUp() throws Exception {
142142
NewPipe.init(DownloaderTestImpl.getInstance());
143-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_SONGS), "");
143+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_SONGS), null);
144144
extractor.fetchPage();
145145
}
146146

@@ -164,7 +164,7 @@ public static class CorrectedSearch extends DefaultSearchExtractorTest {
164164
@BeforeAll
165165
public static void setUp() throws Exception {
166166
NewPipe.init(DownloaderTestImpl.getInstance());
167-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_SONGS), "");
167+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(YoutubeSearchQueryHandlerFactory.MUSIC_SONGS), null);
168168
extractor.fetchPage();
169169
}
170170

extractor/src/test/java/org/schabi/newpipe/extractor/services/youtube/search/YoutubeSearchExtractorTest.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ public static class Channel extends DefaultSearchExtractorTest {
7272
public static void setUp() throws Exception {
7373
YoutubeTestsUtils.ensureStateless();
7474
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "channel"));
75-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(CHANNELS), "");
75+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(CHANNELS), null);
7676
extractor.fetchPage();
7777
}
7878

@@ -96,7 +96,7 @@ public static class Playlists extends DefaultSearchExtractorTest {
9696
public static void setUp() throws Exception {
9797
YoutubeTestsUtils.ensureStateless();
9898
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "playlist"));
99-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(PLAYLISTS), "");
99+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(PLAYLISTS), null);
100100
extractor.fetchPage();
101101
}
102102

@@ -120,7 +120,7 @@ public static class Videos extends DefaultSearchExtractorTest {
120120
public static void setUp() throws Exception {
121121
YoutubeTestsUtils.ensureStateless();
122122
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "videos"));
123-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), "");
123+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), null);
124124
extractor.fetchPage();
125125
}
126126

@@ -146,7 +146,7 @@ public static class Suggestion extends DefaultSearchExtractorTest {
146146
public static void setUp() throws Exception {
147147
YoutubeTestsUtils.ensureStateless();
148148
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "suggestions"));
149-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), "");
149+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), null);
150150
extractor.fetchPage();
151151
}
152152

@@ -170,7 +170,7 @@ public static class CorrectedSearch extends DefaultSearchExtractorTest {
170170
public static void setUp() throws Exception {
171171
YoutubeTestsUtils.ensureStateless();
172172
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "corrected"));
173-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), "");
173+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), null);
174174
extractor.fetchPage();
175175
}
176176

@@ -229,7 +229,7 @@ public static class PagingTest {
229229
public void duplicatedItemsCheck() throws Exception {
230230
YoutubeTestsUtils.ensureStateless();
231231
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "paging"));
232-
final SearchExtractor extractor = YouTube.getSearchExtractor("cirque du soleil", singletonList(VIDEOS), "");
232+
final SearchExtractor extractor = YouTube.getSearchExtractor("cirque du soleil", singletonList(VIDEOS), null);
233233
extractor.fetchPage();
234234

235235
final ListExtractor.InfoItemsPage<InfoItem> page1 = extractor.getInitialPage();
@@ -247,7 +247,7 @@ public static class MetaInfoTest extends DefaultSearchExtractorTest {
247247
public static void setUp() throws Exception {
248248
YoutubeTestsUtils.ensureStateless();
249249
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "metaInfo"));
250-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), "");
250+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), null);
251251
extractor.fetchPage();
252252
}
253253

@@ -283,7 +283,7 @@ public static class ChannelVerified extends DefaultSearchExtractorTest {
283283
public static void setUp() throws Exception {
284284
YoutubeTestsUtils.ensureStateless();
285285
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "verified"));
286-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(CHANNELS), "");
286+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(CHANNELS), null);
287287
extractor.fetchPage();
288288
}
289289

@@ -320,7 +320,7 @@ public static class VideoUploaderAvatar extends DefaultSearchExtractorTest {
320320
public static void setUp() throws Exception {
321321
YoutubeTestsUtils.ensureStateless();
322322
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "video_uploader_avatar"));
323-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), "");
323+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), null);
324324
extractor.fetchPage();
325325
}
326326

@@ -351,7 +351,7 @@ public static class VideoDescription extends DefaultSearchExtractorTest {
351351
public static void setUp() throws Exception {
352352
YoutubeTestsUtils.ensureStateless();
353353
NewPipe.init(DownloaderFactory.getDownloader(RESOURCE_PATH + "video_description"));
354-
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), "");
354+
extractor = YouTube.getSearchExtractor(QUERY, singletonList(VIDEOS), null);
355355
extractor.fetchPage();
356356
}
357357

0 commit comments

Comments
 (0)