test: update --save-signal tests

Signed-off-by: Arkady Gilinsky <[email protected]>

can we simplify the status/exit logic

Author: ark-g

tests/coveragetest.py

@@ -24,7 +24,7 @@
 from collections.abc import Collection, Iterable, Iterator, Mapping, Sequence
 
 import coverage
-from coverage import Coverage
+from coverage import Coverage, env
 from coverage.cmdline import CoverageScript
 from coverage.data import CoverageData
 from coverage.misc import import_local_file
@@ -417,6 +417,9 @@ def run_command(self, cmd: str, *, status: int = 0) -> str:
         Compare with `command_line`.
 
         """
+        if status < 0 and env.LINUX:
+            # Mac properly returns -signal as the exit status. Linux returns 128 + signal.
+            status = 128 - status
         actual_status, output = self.run_command_status(cmd)
         assert actual_status == status
         return output

tests/test_process.py

@@ -731,7 +731,7 @@ def test_save_signal_usr1(self) -> None:
             print('Done and goodbye')
             """)
         covered_lines = 4
-        self.run_command(f"coverage run --save-signal USR1 {test_file}")
+        self.run_command(f"coverage run --save-signal USR1 {test_file}", status=-signal.SIGKILL)
         self.assert_exists(".coverage")
         data = coverage.CoverageData()
         data.read()