Update to allow unlisted tracks

Author: dylanjeffers

api/v1_users_library_tracks.go

@@ -156,11 +156,12 @@ func (app *ApiServer) v1UsersLibraryTracks(c *fiber.Ctx) error {
 		trackIds = append(trackIds, i.ItemID)
 	}
 
-	// get tracks
+	// get tracks - include unlisted tracks since they may be in the library (e.g., purchases)
 	tracks, err := app.queries.FullTracksKeyed(c.Context(), dbv1.FullTracksParams{
 		GetTracksParams: dbv1.GetTracksParams{
-			Ids:  trackIds,
-			MyID: myId,
+			Ids:             trackIds,
+			MyID:            myId,
+			IncludeUnlisted: true,
 		},
 	})
 	if err != nil {

api/v1_users_library_tracks_test.go

@@ -199,3 +199,25 @@ func TestUsersLibraryTracksQuery(t *testing.T) {
 		assert.Contains(t, item.Item.Title, "T1", "All results should match query")
 	}
 }
+
+func TestUsersLibraryTracksMetadataNotNull(t *testing.T) {
+	app := testAppWithFixtures(t)
+	user1Id := trashid.MustEncodeHashID(1)
+
+	var response struct {
+		Data []struct {
+			ItemID int32 `json:"item_id"`
+			Item   any   `json:"item"`
+		} `json:"data"`
+	}
+
+	// Test that all returned tracks have non-null metadata
+	status, _ := testGet(t, app, "/v1/full/users/"+user1Id+"/library/tracks?type=all", &response)
+	assert.Equal(t, 200, status)
+	assert.GreaterOrEqual(t, len(response.Data), 1, "Should have at least one track")
+
+	// Verify all items have non-null metadata
+	for _, item := range response.Data {
+		assert.NotNil(t, item.Item, "Track metadata should not be null for item_id %d", item.ItemID)
+	}
+}