Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fully unit test Dummy Mode #6358

Open
wxtim opened this issue Sep 5, 2024 · 1 comment
Open

Fully unit test Dummy Mode #6358

wxtim opened this issue Sep 5, 2024 · 1 comment
Labels
could be better Not exactly a bug, but not ideal.
Milestone
8.4.x

Comments

@wxtim
Copy link
Member

wxtim commented Sep 5, 2024

The refactoring in #6039 (and #5820) have revealed that the Dummy mode functions in cylc/flow/run_modes/dummy.py are not super well tested.

That same refactoring makes unit testing straightforward.
@wxtim wxtim added the could be better Not exactly a bug, but not ideal. label Sep 5, 2024
@wxtim wxtim added this to the 8.3.x milestone Sep 5, 2024
@oliver-sanders
Copy link
Member

May be made obsolete by #5961

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
could be better Not exactly a bug, but not ideal.
Projects
None yet
Milestone
8.4.x
Development

No branches or pull requests
2 participants
@oliver-sanders @wxtim