Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Oliver Sanders <[email protected]>
  • Loading branch information
wxtim and oliver-sanders committed Jan 17, 2025
1 parent 71e3f59 commit 060a20a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
3 changes: 2 additions & 1 deletion cylc/flow/scripts/show.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@
from cylc.flow.id import Tokens
from cylc.flow.id_cli import parse_ids
from cylc.flow.network.client_factory import get_client
from cylc.flow.run_modes import RunMode
from cylc.flow.task_outputs import TaskOutputs
from cylc.flow.task_state import (
TASK_STATUSES_ORDERED,
Expand Down Expand Up @@ -349,7 +350,7 @@ async def prereqs_and_outputs_query(
attrs.append("runahead")
if (
t_proxy['runtime']['runMode']
and t_proxy['runtime']['runMode'] != 'Live'
and t_proxy['runtime']['runMode'] != RunMode.LIVE.value
):
attrs.append(f"run mode={t_proxy['runtime']['runMode']}")
state_msg = state
Expand Down
3 changes: 1 addition & 2 deletions tests/integration/scripts/test_show.py
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@ async def test_task_instance_state_flows(
]


async def test_mode_changes(flow, scheduler, start, capsys):
async def test_task_run_mode_changes(flow, scheduler, start, capsys):
"""Broadcasting a change of run mode changes run mode shown by cylc show.
"""
opts = SimpleNamespace(
Expand All @@ -300,7 +300,6 @@ async def test_mode_changes(flow, scheduler, start, capsys):
)
schd = scheduler(
flow({'scheduling': {'graph': {'R1': 'a'}}}),
paused_start=True,
run_mode='live'
)

Expand Down

0 comments on commit 060a20a

Please sign in to comment.