Clean up

Tyrrrz · Jul 25, 2024 · 5488c48 · 5488c48
1 parent d4b1422
commit 5488c48
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 10 deletions.
diff --git a/YoutubeExplode.Tests/PlaylistSpecs.cs b/YoutubeExplode.Tests/PlaylistSpecs.cs
@@ -31,7 +31,7 @@ public async Task I_can_get_the_metadata_of_a_playlist()
         playlist
             .Description.Should()
             .Contain("Digital Analytics Fundamentals course on Analytics Academy");
-        playlist.VideosCount.Should().Be(22);
+        playlist.Count.Should().Be(22);
         playlist.Thumbnails.Should().NotBeEmpty();
     }
 
@@ -83,6 +83,7 @@ public async Task I_can_get_the_metadata_of_any_available_playlist(string playli
         playlist.Url.Should().NotBeNullOrWhiteSpace();
         playlist.Title.Should().NotBeNullOrWhiteSpace();
         playlist.Description.Should().NotBeNull();
+        playlist.Count.Should().NotBe(0);
         playlist.Thumbnails.Should().NotBeEmpty();
     }
 

diff --git a/YoutubeExplode/Bridge/IPlaylistData.cs b/YoutubeExplode/Bridge/IPlaylistData.cs
@@ -12,7 +12,7 @@ internal interface IPlaylistData
 
     string? Description { get; }
 
-    int? VideosCount { get; }
+    int? Count { get; }
 
     IReadOnlyList<ThumbnailData> Thumbnails { get; }
 }
diff --git a/YoutubeExplode/Bridge/PlaylistBrowseResponse.cs b/YoutubeExplode/Bridge/PlaylistBrowseResponse.cs
@@ -102,7 +102,7 @@ internal partial class PlaylistBrowseResponse(JsonElement content) : IPlaylistDa
             ?.GetStringOrNull();
 
     [Lazy]
-    public int? VideosCount =>
+    public int? Count =>
         SidebarPrimary
             ?.GetPropertyOrNull("stats")
             ?.EnumerateArrayOrNull()

diff --git a/YoutubeExplode/Bridge/PlaylistNextResponse.cs b/YoutubeExplode/Bridge/PlaylistNextResponse.cs
@@ -35,7 +35,7 @@ internal partial class PlaylistNextResponse(JsonElement content) : IPlaylistData
     public string? Description => null;
 
     [Lazy]
-    public int? VideosCount =>
+    public int? Count =>
         ContentRoot
             ?.GetPropertyOrNull("totalVideosText")
             ?.GetPropertyOrNull("runs")

diff --git a/YoutubeExplode/Playlists/Playlist.cs b/YoutubeExplode/Playlists/Playlist.cs
@@ -12,7 +12,7 @@ public class Playlist(
     string title,
     Author? author,
     string description,
-    int? videosCount,
+    int? count,
     IReadOnlyList<Thumbnail> thumbnails
 ) : IPlaylist
 {
@@ -34,12 +34,12 @@ IReadOnlyList<Thumbnail> thumbnails
     public string Description { get; } = description;
 
     /// <summary>
-    /// Count of total videos.
+    /// Total count of videos included in the playlist.
     /// </summary>
     /// <remarks>
-    /// May be null in case of playlists with infinite videos (e.g. mixes).
+    /// May be null in case of infinite playlists (e.g. auto-generated mixes).
     /// </remarks>
-    public int? VideosCount { get; } = videosCount;
+    public int? Count { get; } = count;
 
     /// <inheritdoc />
     public IReadOnlyList<Thumbnail> Thumbnails { get; } = thumbnails;

diff --git a/YoutubeExplode/Playlists/PlaylistClient.cs b/YoutubeExplode/Playlists/PlaylistClient.cs
@@ -42,7 +42,7 @@ channelId is not null && channelTitle is not null
         // System playlists have no description
         var description = response.Description ?? "";
 
-        var videosCount = response.VideosCount;
+        var count = response.Count;
 
         var thumbnails = response
             .Thumbnails.Select(t =>
@@ -65,7 +65,7 @@ channelId is not null && channelTitle is not null
             })
             .ToArray();
 
-        return new Playlist(playlistId, title, author, description, videosCount, thumbnails);
+        return new Playlist(playlistId, title, author, description, count, thumbnails);
     }
 
     /// <summary>