[Reproducible-commits] [diffoscope] 01/06: Fix PPU detection and associated tests

Jérémy Bobbio lunar at moszumanska.debian.org
Sat Dec 5 23:18:57 UTC 2015


This is an automated email from the git hooks/post-receive script.

lunar pushed a commit to branch master
in repository diffoscope.

commit 939b4b06b96ef6cfb910517070bda690f45f2cd3
Author: Jérémy Bobbio <lunar at debian.org>
Date:   Sat Dec 5 12:07:45 2015 +0000

    Fix PPU detection and associated tests
    
    Identifying PPUs now requires ppudump to be available, so skip the test if
    that's not the case.
---
 diffoscope/comparators/ppu.py | 2 +-
 tests/comparators/test_ppu.py | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/diffoscope/comparators/ppu.py b/diffoscope/comparators/ppu.py
index fbbeb26..ca3aad7 100644
--- a/diffoscope/comparators/ppu.py
+++ b/diffoscope/comparators/ppu.py
@@ -22,7 +22,7 @@
 import os
 import re
 import subprocess
-from diffoscope import tool_required
+from diffoscope import tool_required, logger
 from diffoscope.comparators.binary import File, needs_content
 from diffoscope.comparators.utils import Command
 from diffoscope.difference import Difference
diff --git a/tests/comparators/test_ppu.py b/tests/comparators/test_ppu.py
index bf0acb3..6326946 100644
--- a/tests/comparators/test_ppu.py
+++ b/tests/comparators/test_ppu.py
@@ -43,6 +43,7 @@ def file1():
 def file2():
     return specialize(FilesystemFile(TEST_FILE2_PATH))
 
+ at pytest.mark.skipif(tool_missing('ppudump'), reason='missing ppudump')
 def test_identification(file1):
     assert isinstance(file1, PpuFile)
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/reproducible/diffoscope.git



More information about the Reproducible-commits mailing list