From 4652e47c6516f64d9beb0bc132d53acf437b51ce Mon Sep 17 00:00:00 2001 From: David Sansome Date: Wed, 3 Mar 2010 01:39:39 +0000 Subject: [PATCH] Test adding a song without required fields set --- tests/librarybackend_test.cpp | 37 +++++++++++++++++++++++++++++------ 1 file changed, 31 insertions(+), 6 deletions(-) diff --git a/tests/librarybackend_test.cpp b/tests/librarybackend_test.cpp index c82726a98..8242c32a2 100644 --- a/tests/librarybackend_test.cpp +++ b/tests/librarybackend_test.cpp @@ -109,6 +109,37 @@ TEST_F(LibraryBackendTest, RemoveDirectory) { EXPECT_EQ(1, list[0].id); } +TEST_F(LibraryBackendTest, AddInvalidSong) { + // Adding a song without certain fields set should fail + backend_->AddDirectory("/test"); + Song s; + s.set_directory_id(1); + + QSignalSpy spy(backend_.get(), SIGNAL(Error(QString))); + + backend_->AddOrUpdateSongs(SongList() << s); + ASSERT_EQ(1, spy.count()); spy.takeFirst(); + + s.set_filename("foo"); + backend_->AddOrUpdateSongs(SongList() << s); + ASSERT_EQ(1, spy.count()); spy.takeFirst(); + + s.set_filesize(100); + backend_->AddOrUpdateSongs(SongList() << s); + ASSERT_EQ(1, spy.count()); spy.takeFirst(); + + s.set_mtime(100); + backend_->AddOrUpdateSongs(SongList() << s); + ASSERT_EQ(1, spy.count()); spy.takeFirst(); + + s.set_ctime(100); + backend_->AddOrUpdateSongs(SongList() << s); + ASSERT_EQ(0, spy.count()); +} + +TEST_F(LibraryBackendTest, GetAlbumArtNonExistent) { +} + // Test adding a single song to the database, then getting various information // back about it. @@ -215,9 +246,3 @@ TEST_F(SingleSong, FindSongsInDirectory) { EXPECT_EQ(song_.title(), songs[0].title()); EXPECT_EQ(1, songs[0].id()); } - -TEST_F(LibraryBackendTest, AddSongWithoutFilename) { -} - -TEST_F(LibraryBackendTest, GetAlbumArtNonExistent) { -}