Fix cylc show satisfied xtriggers info

[dwsutherland]

closes #5683

Check List

• I have read CONTRIBUTING.md and added my name as a Code Contributor.
• Contains logically grouped changes (else tidy your branch by rebase).
• Does not contain off-topic changes (use other PRs for other changes).
• Applied any dependency changes to both setup.cfg (and conda-environment.yml if present).
• Tests are included (or explain why tests are not needed).
• If this is a bug fix, PR should be raised against the relevant ?.?.x branch.

[wxtim]

Looks reasonable, and seems to work.

I wonder if Oliver's new test framework in #5731 might allow us to test it. I might have a play - since that'll help with this and be good reviewing of that PR>

[MetRonnie]

Tested, working. Might be useful to add a type annotation here

--- a/cylc/flow/data_store_mgr.py
+++ b/cylc/flow/data_store_mgr.py
@@ -482,5 +484,5 @@ class DataStoreMgr:
             for state in TASK_STATUSES_ORDERED
         }
-        self.xtrigger_tasks = {}
+        self.xtrigger_tasks: Dict[str, Set[Tuple[str, str]]] = {}
         # Managed data types
         self.data = {

[oliver-sanders]

I wonder if Oliver's new test framework in #5731 might allow us to test it

That's just for Tui and at present Tui doesn't support cylc show.

[hjoliver]

Looks good, but might as well add the suggested type annotation @dwsutherland

[dwsutherland]

Might be useful to add a type annotation here

Done.