Skip to content

Commit 8d40f52

Browse files
committed
Remove unused fixtures
1 parent 5524191 commit 8d40f52

File tree

1 file changed

+7
-28
lines changed

1 file changed

+7
-28
lines changed

tests/unit/assessment/test_sequencing.py

Lines changed: 7 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,7 @@ def test_register_job_with_non_existing_cluster(ws, admin_locator) -> None:
5858
]
5959

6060

61-
def test_register_job_with_existing_job_cluster_key(
62-
ws,
63-
mock_path_lookup,
64-
admin_locator,
65-
) -> None:
61+
def test_register_job_with_existing_job_cluster_key(ws, admin_locator) -> None:
6662
"""Register a job with a task referencing a existing job cluster."""
6763
job_cluster = jobs.JobCluster("existing-id", ClusterSpec())
6864
task = jobs.Task(task_key="test-task", job_cluster_key="existing-id")
@@ -75,11 +71,7 @@ def test_register_job_with_existing_job_cluster_key(
7571
assert not maybe_node.failed
7672

7773

78-
def test_register_job_with_non_existing_job_cluster_key(
79-
ws,
80-
mock_path_lookup,
81-
admin_locator,
82-
) -> None:
74+
def test_register_job_with_non_existing_job_cluster_key(ws, admin_locator) -> None:
8375
"""Register a job with a task referencing a non-existing job cluster."""
8476
task = jobs.Task(task_key="test-task", job_cluster_key="non-existing-id")
8577
settings = jobs.JobSettings(name="test-job", tasks=[task])
@@ -98,12 +90,7 @@ def test_register_job_with_non_existing_job_cluster_key(
9890
]
9991

10092

101-
def test_register_job_with_new_cluster(
102-
ws,
103-
simple_dependency_resolver,
104-
mock_path_lookup,
105-
admin_locator,
106-
) -> None:
93+
def test_register_job_with_new_cluster(ws, admin_locator) -> None:
10794
"""Register a job with a task with a new cluster definition."""
10895
task = jobs.Task(task_key="test-task", new_cluster=ClusterSpec())
10996
settings = jobs.JobSettings(name="test-job", tasks=[task])
@@ -116,9 +103,7 @@ def test_register_job_with_new_cluster(
116103
assert not maybe_node.failed
117104

118105

119-
def test_sequence_steps_from_job_task_with_existing_cluster_id(
120-
ws, simple_dependency_resolver, mock_path_lookup, admin_locator
121-
) -> None:
106+
def test_sequence_steps_from_job_task_with_existing_cluster_id(ws, admin_locator) -> None:
122107
"""Sequence a job with a task referencing an existing cluster.
123108
124109
Sequence:
@@ -176,9 +161,7 @@ def get_cluster(cluster_id: str) -> ClusterDetails:
176161
]
177162

178163

179-
def test_sequence_steps_from_job_task_with_existing_job_cluster_key(
180-
ws, simple_dependency_resolver, mock_path_lookup, admin_locator
181-
) -> None:
164+
def test_sequence_steps_from_job_task_with_existing_job_cluster_key(ws, admin_locator) -> None:
182165
"""Sequence a job with a task referencing an existing job cluster.
183166
184167
Sequence:
@@ -226,9 +209,7 @@ def test_sequence_steps_from_job_task_with_existing_job_cluster_key(
226209
]
227210

228211

229-
def test_sequence_steps_from_job_task_with_new_cluster(
230-
ws, simple_dependency_resolver, mock_path_lookup, admin_locator
231-
) -> None:
212+
def test_sequence_steps_from_job_task_with_new_cluster(ws, admin_locator) -> None:
232213
"""Sequence a job with a task that has a new cluster definition.
233214
234215
Sequence:
@@ -265,9 +246,7 @@ def test_sequence_steps_from_job_task_with_new_cluster(
265246
]
266247

267248

268-
def test_sequence_steps_from_job_task_with_non_existing_cluster(
269-
ws, simple_dependency_resolver, mock_path_lookup, admin_locator
270-
) -> None:
249+
def test_sequence_steps_from_job_task_with_non_existing_cluster(ws, admin_locator) -> None:
271250
"""Sequence a job with a task that references a non-existing cluster.
272251
273252
Sequence:

0 commit comments

Comments
 (0)