Merge pull request #47 from pycompression/morecoverage

Add a few extra tests for improved coverage

Author: rhpvorderman

codecov.yml

@@ -1,4 +1,3 @@
-comment: off
 coverage:
   status:
     project:

tests/test_igzip.py

@@ -24,14 +24,19 @@
 
 import gzip
 import io
+import os
+import shutil
 import sys
+import tempfile
+from pathlib import Path
 
 from isal import igzip
 
 import pytest
 
 DATA = b'This is a simple test with igzip'
 COMPRESSED_DATA = gzip.compress(DATA)
+TEST_FILE = str((Path(__file__).parent / "data" / "test.fastq.gz"))
 
 
 def test_wrong_compresslevel_igzipfile():
@@ -40,6 +45,20 @@ def test_wrong_compresslevel_igzipfile():
     error.match("Compression level should be between 0 and 3")
 
 
+def test_repr():
+    tempdir = tempfile.mkdtemp()
+    with igzip.IGzipFile(os.path.join(tempdir, "test.gz"), "wb") as test:
+        assert "<igzip _io.BufferedWriter name='" in repr(test)
+    shutil.rmtree(tempdir)
+
+
+def test_write_readonly_file():
+    with igzip.IGzipFile(TEST_FILE, "rb") as test:
+        with pytest.raises(OSError) as error:
+            test.write(b"bla")
+    error.match(r"write\(\) on read-only IGzipFile object")
+
+
 @pytest.mark.parametrize("level", range(1, 10))
 def test_decompress_stdin_stdout(capsysbinary, level):
     """Test if the command line can decompress data that has been compressed

tox.ini

@@ -9,7 +9,7 @@ passenv=
     PYTHON_ISAL_LINK_DYNAMIC
 commands =
     # Create HTML coverage report for humans and xml coverage report for external services.
-    coverage run --source={envsitepackagesdir}/isal -m py.test tests
+    coverage run --source=isal -m py.test tests
     coverage html
     coverage xml
 
@@ -24,7 +24,7 @@ setenv=
     CYTHON_COVERAGE=true
 commands =
     # Create HTML coverage report for humans and xml coverage report for external services.
-    coverage run --source={envsitepackagesdir}/isal -m py.test tests
+    coverage run --source=isal -m py.test tests
     coverage html
     coverage xml