Funkwhale
/
Clementine-audio-player-Mac-Linux-Windows
mirror of https://github.com/clementine-player/Clementine
1
0
Fork 0
Code Issues Releases Wiki Activity

Add tests to check the tag edition of FLAC/OGG files and make format

This commit is contained in:
Chocobozzz 2015-01-17 12:11:33 +01:00
parent 2a7d4e6d5e
commit b96c1f060f
2 changed files with 115 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
src/ui/edittagdialog.cpp
View File

@ -216,8 +216,8 @@ bool EditTagDialog::SetLoading(const QString& message) {
return true;
}
QList<EditTagDialog::Data> EditTagDialog::LoadData(const SongList& songs)
const {
QList<EditTagDialog::Data> EditTagDialog::LoadData(
const SongList& songs) const {
QList<Data> ret;
for (const Song& song : songs) {
@ -246,17 +246,16 @@ void EditTagDialog::SetSongs(const SongList& s, const PlaylistItemList& items) {
ui_->song_list->clear();
// Reload tags in the background
QFuture<QList<Data> > future =
QFuture<QList<Data>> future =
QtConcurrent::run(this, &EditTagDialog::LoadData, s);
QFutureWatcher<QList<Data> >* watcher =
new QFutureWatcher<QList<Data> >(this);
QFutureWatcher<QList<Data>>* watcher = new QFutureWatcher<QList<Data>>(this);
watcher->setFuture(future);
connect(watcher, SIGNAL(finished()), SLOT(SetSongsFinished()));
}
void EditTagDialog::SetSongsFinished() {
QFutureWatcher<QList<Data> >* watcher =
dynamic_cast<QFutureWatcher<QList<Data> >*>(sender());
QFutureWatcher<QList<Data>>* watcher =
dynamic_cast<QFutureWatcher<QList<Data>>*>(sender());
if (!watcher) return;
watcher->deleteLater();
@ -313,19 +312,32 @@ QVariant EditTagDialog::Data::value(const Song& song, const QString& id) {
}
void EditTagDialog::Data::set_value(const QString& id, const QVariant& value) {
if (id == "title") current_.set_title(value.toString());
else if (id == "artist") current_.set_artist(value.toString());
else if (id == "album") current_.set_album(value.toString());
else if (id == "albumartist") current_.set_albumartist(value.toString());
else if (id == "composer") current_.set_composer(value.toString());
else if (id == "performer") current_.set_performer(value.toString());
else if (id == "grouping") current_.set_grouping(value.toString());
else if (id == "genre") current_.set_genre(value.toString());
else if (id == "comment") current_.set_comment(value.toString());
else if (id == "track") current_.set_track(value.toInt());
else if (id == "disc") current_.set_disc(value.toInt());
else if (id == "year") current_.set_year(value.toInt());
else qLog(Warning) << "Unknown ID" << id;
if (id == "title")
current_.set_title(value.toString());
else if (id == "artist")
current_.set_artist(value.toString());
else if (id == "album")
current_.set_album(value.toString());
else if (id == "albumartist")
current_.set_albumartist(value.toString());
else if (id == "composer")
current_.set_composer(value.toString());
else if (id == "performer")
current_.set_performer(value.toString());
else if (id == "grouping")
current_.set_grouping(value.toString());
else if (id == "genre")
current_.set_genre(value.toString());
else if (id == "comment")
current_.set_comment(value.toString());
else if (id == "track")
current_.set_track(value.toInt());
else if (id == "disc")
current_.set_disc(value.toInt());
else if (id == "year")
current_.set_year(value.toInt());
else
qLog(Warning) << "Unknown ID" << id;
}
bool EditTagDialog::DoesValueVary(const QModelIndexList& sel,
@ -676,7 +688,7 @@ void EditTagDialog::SaveData(const QList<Data>& data) {
if (ref.current_.IsMetadataEqual(ref.original_)) continue;
if (!TagReaderClient::Instance()->SaveFileBlocking(
ref.current_.url().toLocalFile(), ref.current_)) {
ref.current_.url().toLocalFile(), ref.current_)) {
emit Error(tr("An error occurred writing metadata to '%1'")
.arg(ref.current_.url().toLocalFile()));
}

95
tests/song_test.cpp
View File

@ -19,7 +19,7 @@
#include "tagreader.h"
#include "core/song.h"
#ifdef HAVE_LIBLASTFM
#include "internet/lastfm/lastfmcompat.h"
#include "internet/lastfm/lastfmcompat.h"
#endif
#include "gmock/gmock.h"
@ -64,7 +64,8 @@ class SongTest : public ::testing::Test {
tag_reader.SaveFile(filename, pb_song);
}
static void WriteSongStatisticsToFile(const Song& song, const QString& filename) {
static void WriteSongStatisticsToFile(const Song& song,
const QString& filename) {
TagReader tag_reader;
::pb::tagreader::SongMetadata pb_song;
song.ToProtobuf(&pb_song);
@ -79,7 +80,6 @@ class SongTest : public ::testing::Test {
}
};
#ifdef HAVE_LIBLASTFM
TEST_F(SongTest, InitsFromLastFM) {
Song song;
@ -95,7 +95,7 @@ TEST_F(SongTest, InitsFromLastFM) {
EXPECT_EQ("Baz", song.album());
EXPECT_EQ("Bar", song.artist());
}
#endif // HAVE_LIBLASTFM
#endif // HAVE_LIBLASTFM
/*TEST_F(SongTest, InitsFromFile) {
QTemporaryFile temp;
@ -156,14 +156,13 @@ TEST_F(SongTest, FMPSPlayCountBoth) {
TEST_F(SongTest, FMPSUnrated) {
QStringList files_to_test;
files_to_test <<
":/testdata/beep.m4a" <<
":/testdata/beep.mp3" <<
":/testdata/beep.flac" <<
":/testdata/beep.ogg" <<
":/testdata/beep.spx" <<
":/testdata/beep.wav" <<
":/testdata/beep.wma";
files_to_test << ":/testdata/beep.m4a"
<< ":/testdata/beep.mp3"
<< ":/testdata/beep.flac"
<< ":/testdata/beep.ogg"
<< ":/testdata/beep.spx"
<< ":/testdata/beep.wav"
<< ":/testdata/beep.wma";
for (const QString& test_filename : files_to_test) {
TemporaryResource r(test_filename);
Song song = ReadSongFromFile(r.fileName());
@ -240,6 +239,41 @@ TEST_F(SongTest, StatisticsOgg) {
EXPECT_EQ(87, new_song.score());
}
TEST_F(SongTest, TagsOgg) {
TemporaryResource r(":/testdata/beep.ogg");
{
Song song = ReadSongFromFile(r.fileName());
song.set_title("beep title");
song.set_artist("beep artist");
song.set_album("beep album");
song.set_albumartist("beep album artist");
song.set_composer("beep composer");
song.set_performer("beep performer");
song.set_grouping("beep grouping");
song.set_genre("beep genre");
song.set_comment("beep comment");
song.set_track(12);
song.set_disc(1234);
song.set_year(2015);
WriteSongToFile(song, r.fileName());
}
Song new_song = ReadSongFromFile(r.fileName());
EXPECT_EQ("beep title", new_song.title());
EXPECT_EQ("beep artist", new_song.artist());
EXPECT_EQ("beep album", new_song.album());
EXPECT_EQ("beep album artist", new_song.albumartist());
EXPECT_EQ("beep composer", new_song.composer());
EXPECT_EQ("beep performer", new_song.performer());
EXPECT_EQ("beep grouping", new_song.grouping());
EXPECT_EQ("beep genre", new_song.genre());
EXPECT_EQ("beep comment", new_song.comment());
EXPECT_EQ(12, new_song.track());
EXPECT_EQ(1234, new_song.disc());
EXPECT_EQ(2015, new_song.year());
}
TEST_F(SongTest, RatingFLAC) {
TemporaryResource r(":/testdata/beep.flac");
{
@ -267,6 +301,41 @@ TEST_F(SongTest, StatisticsFLAC) {
EXPECT_EQ(87, new_song.score());
}
TEST_F(SongTest, TagsFLAC) {
TemporaryResource r(":/testdata/beep.flac");
{
Song song = ReadSongFromFile(r.fileName());
song.set_title("beep title");
song.set_artist("beep artist");
song.set_album("beep album");
song.set_albumartist("beep album artist");
song.set_composer("beep composer");
song.set_performer("beep performer");
song.set_grouping("beep grouping");
song.set_genre("beep genre");
song.set_comment("beep comment");
song.set_track(12);
song.set_disc(1234);
song.set_year(2015);
WriteSongToFile(song, r.fileName());
}
Song new_song = ReadSongFromFile(r.fileName());
EXPECT_EQ("beep title", new_song.title());
EXPECT_EQ("beep artist", new_song.artist());
EXPECT_EQ("beep album", new_song.album());
EXPECT_EQ("beep album artist", new_song.albumartist());
EXPECT_EQ("beep composer", new_song.composer());
EXPECT_EQ("beep performer", new_song.performer());
EXPECT_EQ("beep grouping", new_song.grouping());
EXPECT_EQ("beep genre", new_song.genre());
EXPECT_EQ("beep comment", new_song.comment());
EXPECT_EQ(12, new_song.track());
EXPECT_EQ(1234, new_song.disc());
EXPECT_EQ(2015, new_song.year());
}
#ifdef TAGLIB_WITH_ASF
TEST_F(SongTest, RatingASF) {
TemporaryResource r(":/testdata/beep.wma");
@ -295,7 +364,7 @@ TEST_F(SongTest, StatisticsASF) {
EXPECT_EQ(1337, new_song.playcount());
EXPECT_EQ(87, new_song.score());
}
#endif // TAGLIB_WITH_ASF
#endif // TAGLIB_WITH_ASF
TEST_F(SongTest, RatingMP4) {
TemporaryResource r(":/testdata/beep.m4a");