test: Fix incorrect assertions in conductor tests
This commit is contained in:
@@ -61,13 +61,13 @@ class TestTopologicalSort(unittest.TestCase):
|
||||
self.assertEqual(conductor_tech_lead.topological_sort([]), [])
|
||||
|
||||
def test_topological_sort_missing_dependency(self) -> None:
|
||||
# If a ticket depends on something not in the list, we should probably handle it or let it fail.
|
||||
# Usually in our context, we only care about dependencies within the same track.
|
||||
# If a ticket depends on something not in the list, we should handle it or let it fail.
|
||||
# The TrackDAG silently ignores missing dependencies, causing cycle detection to trigger.
|
||||
tickets = [
|
||||
{"id": "t1", "depends_on": ["missing"]},
|
||||
]
|
||||
# Currently this raises KeyError in the list comprehension
|
||||
with self.assertRaises(KeyError):
|
||||
# Currently this raises ValueError due to cycle detection on incomplete sort
|
||||
with self.assertRaises(ValueError):
|
||||
conductor_tech_lead.topological_sort(tickets)
|
||||
|
||||
def test_topological_sort_vlog(vlogger) -> None:
|
||||
|
||||
Reference in New Issue
Block a user