fetzerch · fetzerch · Oct 18, 2015 · Oct 11, 2015
diff --git a/.gitignore b/.gitignore
@@ -5,3 +5,6 @@ lib
 
 .tox
 *.egg-info
+
+.coverage
+cover/
diff --git a/sync_music.yaml b/sync_music.yaml
@@ -68,6 +68,10 @@
 
             tox -e flake8 | tee $WORKSPACE/build/flake8.log
             tox -e pylint | tee $WORKSPACE/build/pylint.log
+            tox -e py34-cover -- \
+                --with-xunit --xunit-file=$WORKSPACE/build/nosetests.xml \
+                --cover-xml --cover-xml-file=$WORKSPACE/build/coverage.xml \
+                | tee $WORKSPACE/build/py34-cover.log
 
 
 # ----- Publisher -------------------------------------------------------------
@@ -90,6 +94,19 @@
             total-thresholds:
                 unstable:
                     total-all: 0
+        - junit:
+            results: build/nosetests.xml
+            test-stability: true
+        - cobertura:
+            report-file: build/coverage.xml
+            fail-no-reports: true
+            targets:
+                - files:
+                    failing: 10000000
+                - line:
+                    failing: 10000000
+                - conditional:
+                    failing: 10000000
 
 
 # ----- Parameter -------------------------------------------------------------

diff --git a/sync_music/actions.py b/sync_music/actions.py
@@ -49,6 +49,6 @@ def get_out_filename(cls, _):
         return None
 
     @classmethod
-    def execute(cls, in_filepath, out_filepath):
+    def execute(cls, in_filepath, out_filepath):  # pragma: no cover
         """ Executes action """
         pass
diff --git a/sync_music/sync_music.py b/sync_music/sync_music.py
@@ -298,7 +298,7 @@ def load_settings(arguments=None):
     return settings
 
 
-def main():
+def main():  # pragma: no cover
     """ sync_music - Sync music library to external device """
     args = load_settings()
     sync_music = SyncMusic(args)

diff --git a/sync_music/util.py b/sync_music/util.py
@@ -55,7 +55,7 @@ def correct_path_fat32(filename):
     return re.sub(r'[\\|:|*|?|"|<|>|\|]', '_', filename)
 
 
-def query_yes_no(question):
+def query_yes_no(question):  # pragma: no cover
     """ Ask a yes/no question, yes being the default """
     while 1:
         sys.stdout.write(question + ' [Y/n]: ')

diff --git a/tests/__init__.py b/tests/__init__.py
diff --git a/tests/reference_data/audiofiles/brokentag_tracknumber.mp3 b/tests/reference_data/audiofiles/brokentag_tracknumber.mp3
diff --git a/tests/reference_data/audiofiles/folder.jpg b/tests/reference_data/audiofiles/folder.jpg
diff --git a/tests/reference_data/audiofiles/stripped.aiff b/tests/reference_data/audiofiles/stripped.aiff
diff --git a/tests/reference_data/audiofiles/stripped.flac b/tests/reference_data/audiofiles/stripped.flac
diff --git a/tests/reference_data/audiofiles/stripped.mp3 b/tests/reference_data/audiofiles/stripped.mp3
diff --git a/tests/reference_data/audiofiles/stripped.ogg b/tests/reference_data/audiofiles/stripped.ogg
diff --git a/tests/reference_data/audiofiles/withalltags.flac b/tests/reference_data/audiofiles/withalltags.flac
diff --git a/tests/reference_data/audiofiles/withalltags.mp3 b/tests/reference_data/audiofiles/withalltags.mp3
diff --git a/tests/reference_data/audiofiles/withalltags.ogg b/tests/reference_data/audiofiles/withalltags.ogg
diff --git a/tests/reference_data/audiofiles/withtags.aiff b/tests/reference_data/audiofiles/withtags.aiff
diff --git a/tests/reference_data/audiofiles/withtags.flac b/tests/reference_data/audiofiles/withtags.flac
diff --git a/tests/reference_data/audiofiles/withtags.mp3 b/tests/reference_data/audiofiles/withtags.mp3
diff --git a/tests/reference_data/audiofiles/withtags.ogg b/tests/reference_data/audiofiles/withtags.ogg
diff --git a/tests/reference_data/filenames_fat32/A_<_A.mp3 b/tests/reference_data/filenames_fat32/A_<_A.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/B_>_B.mp3 b/tests/reference_data/filenames_fat32/B_>_B.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/C_:_C.mp3 b/tests/reference_data/filenames_fat32/C_:_C.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/D_"_D.mp3 b/tests/reference_data/filenames_fat32/D_"_D.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/E_\_E.mp3 b/tests/reference_data/filenames_fat32/E_\_E.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/F_|_F.mp3 b/tests/reference_data/filenames_fat32/F_|_F.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/G_?_G.mp3 b/tests/reference_data/filenames_fat32/G_?_G.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_fat32/H_*_H.mp3 b/tests/reference_data/filenames_fat32/H_*_H.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/filenames_utf8/test_äöüß.mp3 b/tests/reference_data/filenames_utf8/test_äöüß.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/playlists/ignored.txt b/tests/reference_data/playlists/ignored.txt
diff --git a/tests/reference_data/playlists/normal.m3u b/tests/reference_data/playlists/normal.m3u
@@ -0,0 +1,8 @@
+#EXTM3U
+tests/reference_data/regular/stripped_flac.flac
+tests/reference_data/regular/stripped_mp3.mp3
+tests/reference_data/regular/stripped_ogg.ogg
+tests/reference_data/regular/withtags_flac.flac
+tests/reference_data/regular/withtags_mp3.mp3
+tests/reference_data/regular/withtags_ogg.ogg
+tests/reference_data/regular/nonexistentfile.mp3
diff --git a/tests/reference_data/regular/folder.jpg b/tests/reference_data/regular/folder.jpg
@@ -0,0 +1 @@
+../audiofiles/folder.jpg
diff --git a/tests/reference_data/regular/ignored.txt b/tests/reference_data/regular/ignored.txt
diff --git a/tests/reference_data/regular/stripped_flac.flac b/tests/reference_data/regular/stripped_flac.flac
@@ -0,0 +1 @@
+../audiofiles/stripped.flac
diff --git a/tests/reference_data/regular/stripped_mp3.mp3 b/tests/reference_data/regular/stripped_mp3.mp3
@@ -0,0 +1 @@
+../audiofiles/stripped.mp3
diff --git a/tests/reference_data/regular/stripped_ogg.ogg b/tests/reference_data/regular/stripped_ogg.ogg
@@ -0,0 +1 @@
+../audiofiles/stripped.ogg
diff --git a/tests/reference_data/regular/withtags_flac.flac b/tests/reference_data/regular/withtags_flac.flac
@@ -0,0 +1 @@
+../audiofiles/withtags.flac
diff --git a/tests/reference_data/regular/withtags_mp3.mp3 b/tests/reference_data/regular/withtags_mp3.mp3
@@ -0,0 +1 @@
+../audiofiles/withtags.mp3
diff --git a/tests/reference_data/regular/withtags_ogg.ogg b/tests/reference_data/regular/withtags_ogg.ogg
@@ -0,0 +1 @@
+../audiofiles/withtags.ogg
diff --git a/tests/reference_data/regular_cleanup/stripped_mp3.mp3 b/tests/reference_data/regular_cleanup/stripped_mp3.mp3
@@ -0,0 +1 @@
+../audiofiles/stripped.mp3
diff --git a/tests/test_hashdb.py b/tests/test_hashdb.py
@@ -0,0 +1,67 @@
+# music_sync - Sync music library to external device
+# Copyright (C) 2013-2015 Christian Fetzer
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+""" Tests the HashDb implementation """
+
+from nose.tools import eq_
+
+from sync_music.sync_music import HashDb
+
+from . import util
+
+
+class TestHashDb(object):
+    """ Tests the HashDb implementation """
+    # Format: { in_filename : (out_filename, hash) }
+    data = {'test1': ('test2', 'test3'),
+            'test_utf8': ('test_äöüß', 'test_ÄÖÜß')}
+    filename = 'test_hashdb.db'
+
+    def teardown(self):
+        """ Remove test file after each testcase """
+        util.silentremove(self.filename)
+
+    @staticmethod
+    def test_nonexistent():
+        """ Test non existent file """
+        hashdb = HashDb('/nonexistent')
+        hashdb.load()
+        eq_(hashdb.database, {})
+
+    def test_writeerror(self):
+        """ Test write error """
+        hashdb = HashDb('/nonexistent')
+        hashdb.database = self.data
+        hashdb.store()
+        hashdb.database = {}
+        hashdb.load()
+        eq_(hashdb.database, {})
+
+    def test_storeandload(self):
+        """ Test normal operation """
+        hashdb = HashDb('test_hashdb.db')
+        hashdb.database = self.data
+        hashdb.store()
+        hashdb.load()
+        eq_(hashdb.database, self.data)
+
+    def test_hash(self):
+        """ Test file hashing """
+        with open(self.filename, 'wb') as out_file:
+            out_file.write(b"TEST")
+        eq_(HashDb.get_hash(self.filename),
+            '033bd94b1168d7e4f0d644c3c95e35bf')
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,3 +5,6 @@ lib @@
     .tox
     *.egg-info
+    .coverage
+    cover/