mirror of
https://github.com/clementine-player/Clementine
synced 2024-12-24 00:31:36 +01:00
Oops - it helps if we actually remember to set the URL of the song we just loaded. Fixes issue #426
This commit is contained in:
parent
31a0a89452
commit
80837ce652
@ -328,6 +328,7 @@ void SongLoader::StopTypefind() {
|
|||||||
Song song;
|
Song song;
|
||||||
song.set_valid(true);
|
song.set_valid(true);
|
||||||
song.set_filetype(Song::Type_Stream);
|
song.set_filetype(Song::Type_Stream);
|
||||||
|
song.set_filename(url_.toString());
|
||||||
song.set_title(url_.toString());
|
song.set_title(url_.toString());
|
||||||
songs_ << song;
|
songs_ << song;
|
||||||
}
|
}
|
||||||
|
@ -122,6 +122,7 @@ TEST_F(SongLoaderTest, LoadRemoteMp3) {
|
|||||||
|
|
||||||
// Check the song got loaded
|
// Check the song got loaded
|
||||||
ASSERT_EQ(1, loader_->songs().count());
|
ASSERT_EQ(1, loader_->songs().count());
|
||||||
|
EXPECT_EQ(QString(kRemoteUrl) + "/beep.mp3", loader_->songs()[0].filename());
|
||||||
}
|
}
|
||||||
|
|
||||||
TEST_F(SongLoaderTest, LoadRemote404) {
|
TEST_F(SongLoaderTest, LoadRemote404) {
|
||||||
|
Loading…
Reference in New Issue
Block a user