Merge pull request #32 from cucapra/bin-outputs

Allow binary output files

CHANGELOG.md

@@ -6,6 +6,7 @@ Turnt Changelog
 
 - You can now configure `todo` tests to ignore failures.
 - Fix the reported test count when running multiple environments.
+- Fix a crash when a test command outputs binary data.
 
 1.11.0 (2023-07-08)
 -------------------

test/binary/bin.out

@@ -1 +1 @@
-bin
+����

test/binary/turnt.toml

@@ -1,2 +1,2 @@
-command = "echo {base} {args}"
+command = "cat {filename}"
 binary = true

turnt/run.py

@@ -52,10 +52,10 @@ def check_result(cfg: Config, test: Test,
                            stdout=sys.stderr.buffer)
 
         # Read actual & expected output.
-        with open(output_file) as f:
+        with open(output_file, 'rb') as f:
             actual = f.read()
         if os.path.isfile(saved_file):
-            with open(saved_file) as f:
+            with open(saved_file, 'rb') as f:
                 expected = f.read()
         else:
             expected = None