Update test_coincidence.py

cta-observatory · Oct 20, 2023 · 6e083f0 · 6e083f0
1 parent 85b393f
commit 6e083f0
Showing 1 changed file with 4 additions and 13 deletions.
diff --git a/magicctapipe/scripts/lst1_magic/tests/test_coincidence.py b/magicctapipe/scripts/lst1_magic/tests/test_coincidence.py
@@ -1,5 +1,4 @@
 import subprocess
-import pytest
 import glob
 
 
@@ -9,7 +8,7 @@ def test_coincidence_preoffset(dl1_lst, merge_magic, temp_coinc_preoff, config_p
     """
 
     for file in dl1_lst:
-        out = subprocess.run(
+        _ = subprocess.run(
             [
                 "lst1_magic_event_coincidence",
                 f"-l{str(file)}",
@@ -18,11 +17,7 @@ def test_coincidence_preoffset(dl1_lst, merge_magic, temp_coinc_preoff, config_p
                 f"-c{str(config_preoff)}",
             ]
         )
-        if out.returncode != 0:
-            raise ValueError(
-                f"Coincidence script with pre-offset option failed with return code {out.returncode} for file {file}"
-            )    
-
+
     assert len(glob.glob(f"{temp_coinc_preoff}/*.h5"))==1
 
 
@@ -32,7 +27,7 @@ def test_coincidence(dl1_lst, merge_magic, temp_coinc, config):
     """
 
     for file in dl1_lst:
-        out = subprocess.run(
+        _ = subprocess.run(
             [
                 "lst1_magic_event_coincidence",
                 f"-l{str(file)}",
@@ -41,11 +36,7 @@ def test_coincidence(dl1_lst, merge_magic, temp_coinc, config):
                 f"-c{str(config)}",
             ]
         )
-        if out.returncode != 0:
-            raise ValueError(
-                f"Coincidence script failed with return code {out.returncode} for file {file}"
-            )    
-
+
     assert len(glob.glob(f"{temp_coinc}/*.h5"))==1