Skip to content

Commit 0e1b4bb

Browse files
committed
[SoundCloud] Test playlists banner: it should not exist
1 parent 65bdb3b commit 0e1b4bb

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -94,10 +94,10 @@ public void testThumbnailUrl() {
9494
assertIsSecureUrl(extractor.getThumbnailUrl());
9595
}
9696

97-
@Ignore
9897
@Test
9998
public void testBannerUrl() {
100-
assertIsSecureUrl(extractor.getBannerUrl());
99+
// SoundCloud playlists do not have a banner
100+
assertNull(extractor.getBannerUrl());
101101
}
102102

103103
@Test
@@ -186,10 +186,10 @@ public void testThumbnailUrl() {
186186
assertIsSecureUrl(extractor.getThumbnailUrl());
187187
}
188188

189-
@Ignore("not implemented")
190189
@Test
191190
public void testBannerUrl() {
192-
assertIsSecureUrl(extractor.getBannerUrl());
191+
// SoundCloud playlists do not have a banner
192+
assertNull(extractor.getBannerUrl());
193193
}
194194

195195
@Test
@@ -293,10 +293,10 @@ public void testThumbnailUrl() {
293293
assertIsSecureUrl(extractor.getThumbnailUrl());
294294
}
295295

296-
@Ignore
297296
@Test
298297
public void testBannerUrl() {
299-
assertIsSecureUrl(extractor.getBannerUrl());
298+
// SoundCloud playlists do not have a banner
299+
assertNull(extractor.getBannerUrl());
300300
}
301301

302302
@Test

0 commit comments

Comments
 (0)