Skip to content

Commit

Permalink
fix lint issue and comment out monitor test for now
Browse files Browse the repository at this point in the history
  • Loading branch information
bgunnar5 committed Feb 19, 2025
1 parent 3932ba2 commit fc9a8e7
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion tests/fixture_data_classes.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ class ChordErrorSetup:
class MonitorAutoRestartSetup:
"""
TODO fix this docstring!!!
Data class to encapsulate all chord-error-related fixtures required
for testing the chord error workflow.
Expand Down
20 changes: 10 additions & 10 deletions tests/integration/workflows/test_monitor_auto_restart.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@ class TestMonitorAutoRestart:
Tests for the chord error workflow.
"""

def test_monitor_auto_restarts(self, monitor_auto_restart_setup: MonitorAutoRestartSetup):
"""
Test that the monitor automatically restarts the workflow when:
1. There are no tasks in the queues
2. There are no workers processing tasks
3. The workflow has not yet finished
# def test_monitor_auto_restarts(self, monitor_auto_restart_setup: MonitorAutoRestartSetup):
# """
# Test that the monitor automatically restarts the workflow when:
# 1. There are no tasks in the queues
# 2. There are no workers processing tasks
# 3. The workflow has not yet finished

Args:
monitor_auto_restart_setup: A fixture that returns a
[`MonitorAutoRestartSetup`][fixture_data_classes.MonitorAutoRestartSetup] instance.
"""
# Args:
# monitor_auto_restart_setup: A fixture that returns a
# [`MonitorAutoRestartSetup`][fixture_data_classes.MonitorAutoRestartSetup] instance.
# """

0 comments on commit fc9a8e7

Please sign in to comment.