DM-27835 pipetask run should always print stack trace and exception details

python/lsst/obs/base/cli/cmd/commands.py

@@ -33,7 +33,6 @@
                                      )
 from lsst.daf.butler.cli.utils import (
     ButlerCommand,
-    cli_handle_exception,
     split_commas,
     typeStrAcceptsMultiple
 )
@@ -75,7 +74,7 @@ def convert(*args, **kwargs):
     has no chained reruns.  Custom scripts that call ConvertRepoTask should be
     used on more complex suites of repositories.
     """
-    cli_handle_exception(script.convert, *args, **kwargs)
+    script.convert(*args, **kwargs)
 
 
 @click.command(short_help="Define visits from exposures.", cls=ButlerCommand)
@@ -92,7 +91,7 @@ def convert(*args, **kwargs):
 @options_file_option()
 def define_visits(*args, **kwargs):
     """Define visits from exposures in the butler registry."""
-    cli_handle_exception(script.defineVisits, *args, **kwargs)
+    script.defineVisits(*args, **kwargs)
 
 
 @click.command(short_help="Ingest raw frames.", cls=ButlerCommand)
@@ -112,7 +111,7 @@ def define_visits(*args, **kwargs):
 @options_file_option()
 def ingest_raws(*args, **kwargs):
     """Ingest raw frames into from a directory into the butler registry"""
-    cli_handle_exception(script.ingestRaws, *args, **kwargs)
+    script.ingestRaws(*args, **kwargs)
 
 
 @click.command(short_help="Add an instrument to the repository", cls=ButlerCommand)
@@ -121,7 +120,7 @@ def ingest_raws(*args, **kwargs):
 def register_instrument(*args, **kwargs):
     """Add an instrument to the data repository.
     """
-    cli_handle_exception(script.registerInstrument, *args, **kwargs)
+    script.registerInstrument(*args, **kwargs)
 
 
 @click.command(short_help="Add an instrument's curated calibrations.", cls=ButlerCommand)
@@ -136,4 +135,4 @@ def register_instrument(*args, **kwargs):
 def write_curated_calibrations(*args, **kwargs):
     """Add an instrument's curated calibrations to the data repository.
     """
-    cli_handle_exception(script.writeCuratedCalibrations, *args, **kwargs)
+    script.writeCuratedCalibrations(*args, **kwargs)

tests/test_cliCmdConvert.py

@@ -30,6 +30,8 @@
 
 class ConvertTestCase(CliCmdTestBase, unittest.TestCase):
 
+    mockFuncName = "lsst.obs.base.cli.cmd.commands.script.convert"
+
     @staticmethod
     def defaultExpected():
         return dict(skymap_name=None,

tests/test_cliCmdDefineVisits.py

@@ -30,6 +30,8 @@
 
 class DefineVisitsTest(CliCmdTestBase, unittest.TestCase):
 
+    mockFuncName = "lsst.obs.base.cli.cmd.commands.script.defineVisits"
+
     @staticmethod
     def defaultExpected():
         return dict(config_file=None,

tests/test_cliCmdRegisterInstrument.py

@@ -30,6 +30,8 @@
 
 class RegisterInstrumentTest(CliCmdTestBase, unittest.TestCase):
 
+    mockFuncName = "lsst.obs.base.cli.cmd.commands.script.registerInstrument"
+
     @staticmethod
     def defaultExpected():
         return dict()

tests/test_cliCmdTestIngest.py

@@ -36,6 +36,8 @@
 
 class IngestRawsTestCase(CliCmdTestBase, unittest.TestCase):
 
+    mockFuncName = "lsst.obs.base.cli.cmd.commands.script.ingestRaws"
+
     @staticmethod
     def defaultExpected():
         return dict(config={},

tests/test_cliCmdWriteCuratedCalibrations.py

@@ -30,6 +30,8 @@
 
 class WriteCuratedCalibrationsTest(CliCmdTestBase, unittest.TestCase):
 
+    mockFuncName = "lsst.obs.base.cli.cmd.commands.script.writeCuratedCalibrations"
+
     @staticmethod
     def defaultExpected():
         return dict()