Skip to content

Commit 9831bcc

Browse files
committed
Add more tests for invalid channel URL patterns
1 parent 75a4ccb commit 9831bcc

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

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

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,12 +53,18 @@ void acceptUrlTest() throws ParsingException {
5353

5454
// do not accept URLs which are not channels
5555
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/watch?v=jZViOEv90dI&t=100"));
56+
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/watch"));
5657
assertFalse(linkHandler.acceptUrl("http://www.youtube.com/watch_popup?v=uEJuoEs1UxY"));
58+
assertFalse(linkHandler.acceptUrl("http://www.youtube.com/watch_popup"));
5759
assertFalse(linkHandler.acceptUrl("http://www.youtube.com/attribution_link?a=JdfC0C9V6ZI&u=%2Fwatch%3Fv%3DEhxJLojIE_o%26feature%3Dshare"));
60+
assertFalse(linkHandler.acceptUrl("http://www.youtube.com/attribution_link"));
5861
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/playlist?list=PLW5y1tjAOzI3orQNF1yGGVL5x-pR2K1d"));
62+
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/playlist"));
5963
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/embed/jZViOEv90dI"));
64+
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/embed"));
6065
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/feed/subscriptions?list=PLz8YL4HVC87WJQDzVoY943URKQCsHS9XV"));
61-
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/?app=desktop&persist_app=1"));
66+
assertFalse(linkHandler.acceptUrl("https://www.youtube.com/feed"));
67+
assertFalse(linkHandler.acceptUrl("https://www.youtube.com"));
6268
assertFalse(linkHandler.acceptUrl("https://m.youtube.com/select_site"));
6369
}
6470

0 commit comments

Comments
 (0)