test(lidarr): give the download record fixture distinct album and artist ids
The shared download record held id, album id, and artist id all equal to 1, so reading the wrong one of the three could not fail any test that uses the fixture. Move artist id to 3 and id to 9, keeping album id at 1 so the fully populated album view still shows its track as downloading, and update the three handler assertions that named the old id.
This commit is contained in:
2 files changed
+5
-5
No files matched your search
@@ -141,7 +141,7 @@ mod tests {
|
||||
#[case(
|
||||
ActiveLidarrBlock::Downloads,
|
||||
ActiveLidarrBlock::DeleteDownloadPrompt,
|
||||
LidarrEvent::DeleteDownload(1)
|
||||
LidarrEvent::DeleteDownload(9)
|
||||
)]
|
||||
#[case(
|
||||
ActiveLidarrBlock::Downloads,
|
||||
@@ -338,7 +338,7 @@ mod tests {
|
||||
#[case(
|
||||
ActiveLidarrBlock::Downloads,
|
||||
ActiveLidarrBlock::DeleteDownloadPrompt,
|
||||
LidarrEvent::DeleteDownload(1)
|
||||
LidarrEvent::DeleteDownload(9)
|
||||
)]
|
||||
#[case(
|
||||
ActiveLidarrBlock::Downloads,
|
||||
@@ -423,7 +423,7 @@ mod tests {
|
||||
)
|
||||
.extract_download_id();
|
||||
|
||||
assert_eq!(download_id, 1);
|
||||
assert_eq!(download_id, 9);
|
||||
}
|
||||
|
||||
#[test]
|
||||
|
||||
@@ -184,9 +184,9 @@ pub mod test_utils {
|
||||
DownloadRecord {
|
||||
title: "Test download title".to_owned(),
|
||||
status: DownloadStatus::Downloading,
|
||||
id: 1,
|
||||
id: 9,
|
||||
album_id: Some(Number::from(1i64)),
|
||||
artist_id: Some(Number::from(1i64)),
|
||||
artist_id: Some(Number::from(3i64)),
|
||||
size: 3543348019f64,
|
||||
sizeleft: 1771674009f64,
|
||||
output_path: Some(HorizontallyScrollableText::from("/nfs/music/alex/album")),
|
||||
|
||||
Reference in new issue
Block a user