Skip to content

Commit

Permalink
Merge pull request #3607 from ThomasWaldmann/fix-zstd-testing
Browse files Browse the repository at this point in the history
fix erroneously skipped zstd compressor tests, fixes #3606
  • Loading branch information
ThomasWaldmann committed Feb 10, 2018
2 parents 24b0bcb + 246a8d5 commit 1922ae3
Showing 1 changed file with 3 additions and 13 deletions.
16 changes: 3 additions & 13 deletions src/borg/testsuite/compress.py
Expand Up @@ -5,11 +5,6 @@
except ImportError:
lzma = None

try:
import zstd
except ImportError:
zstd = None

import pytest

from ..compress import get_compressor, Compressor, CompressionSpec, CNONE, ZLIB, LZ4, LZMA, ZSTD, Auto
Expand Down Expand Up @@ -75,8 +70,6 @@ def test_lzma():


def test_zstd():
if zstd is None:
pytest.skip("No zstd support found.")
c = get_compressor(name='zstd')
cdata = c.compress(data)
assert len(cdata) < len(data)
Expand Down Expand Up @@ -109,6 +102,9 @@ def test_compressor():
params_list = [
dict(name='none'),
dict(name='lz4'),
dict(name='zstd', level=1),
dict(name='zstd', level=3),
# avoiding high zstd levels, memory needs unclear
dict(name='zlib', level=0),
dict(name='zlib', level=6),
dict(name='zlib', level=9),
Expand All @@ -119,12 +115,6 @@ def test_compressor():
dict(name='lzma', level=6),
# we do not test lzma on level 9 because of the huge memory needs
]
if zstd:
params_list += [
dict(name='zstd', level=1),
dict(name='zstd', level=3),
# also avoiding high zstd levels, memory needs unclear
]
for params in params_list:
c = Compressor(**params)
assert data == c.decompress(c.compress(data))
Expand Down

0 comments on commit 1922ae3

Please sign in to comment.