Merge pull request #222 from lsst/tickets/DM-38081

DM-38081: Fix for coverage option passed to run_qbb
lsst · Feb 21, 2023 · 1d38f3a · 1d38f3a
2 parents b50b1b1 + 87d485a
commit 1d38f3a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/python/lsst/ctrl/mpexec/cli/cmd/commands.py b/python/lsst/ctrl/mpexec/cli/cmd/commands.py
@@ -175,8 +175,8 @@ def qgraph(ctx: click.Context, **kwargs: Any) -> None:
     """Build and optionally save quantum graph."""
     kwargs = _collectActions(ctx, **kwargs)
     coverage = kwargs.pop("coverage", False)
+    coverage_packages = kwargs.pop("cov_packages", ())
     if coverage:
-        coverage_packages = kwargs.pop("cov_packages", ())
         cov = _start_coverage(coverage_packages)
 
     try:
@@ -308,8 +308,8 @@ def run_qbb(repo: str, qgraph: str, **kwargs: Any) -> None:
     QGRAPH is the path to a serialized Quantum Graph file.
     """
     coverage = kwargs.pop("coverage", False)
+    coverage_packages = kwargs.pop("cov_packages", ())
     if coverage:
-        coverage_packages = kwargs.pop("cov_packages", ())
         cov = _start_coverage(coverage_packages)
 
     try: