Skip to content

Commit

Permalink
Merge pull request #261 from lsst/tickets/DM-40392
Browse files Browse the repository at this point in the history
DM-40392: Update unit test after fixing QuantumGraph.updateRun
  • Loading branch information
andy-slac authored Aug 19, 2023
2 parents 0ca651f + 4b0fd74 commit 3ef28a6
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions tests/test_cmdLineFwk.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@
Registry,
)
from lsst.daf.butler.core.datasets.type import DatasetType
from lsst.daf.butler.registry import ConflictingDefinitionError, RegistryConfig
from lsst.daf.butler.registry import RegistryConfig
from lsst.pipe.base import (
Instrument,
Pipeline,
Expand Down Expand Up @@ -590,15 +590,8 @@ def test_simple_qgraph_qbb(self):
fwk.runGraphQBB(taskFactory, args)

# Transfer the datasets to the butler.
# TODO: DM-40392
# This will fail because the UUIDs are not updated by updateRun
with self.assertRaises(ConflictingDefinitionError):
n2 = transfer_from_graph(temp_graph.name, self.root, True, False, False)

# For reasons that have to be investigated the number of
# outputs the second time around is only 21.
self.assertEqual(n2, 31)
self.assertEqual(n1, n2)
n2 = transfer_from_graph(temp_graph.name, self.root, True, False, False)
self.assertEqual(n1, n2)

def testEmptyQGraph(self):
"""Test that making an empty QG produces the right error messages."""
Expand Down

0 comments on commit 3ef28a6

Please sign in to comment.