Skip to content

Commit 1ce4749

Browse files
committed
fix test
1 parent 54b40c0 commit 1ce4749

File tree

1 file changed

+13
-6
lines changed

1 file changed

+13
-6
lines changed

services/api-server/tests/unit/test_services_solver_job_models_converters.py

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,13 @@ def test_create_project_model_for_job(faker: Faker):
5353
)
5454

5555
# body of create project!
56-
createproject_body = create_new_project_for_job(solver, job, inputs)
56+
createproject_body = create_new_project_for_job(
57+
solver_or_program=solver,
58+
job=job,
59+
inputs=inputs,
60+
description=None,
61+
name=None,
62+
)
5763

5864
# ensures one-to-one relation
5965
assert createproject_body.uuid == job.id
@@ -216,15 +222,16 @@ def fake_url_for(*args, **kwargs) -> HttpUrl:
216222
)
217223

218224
assert job.id == project.uuid
219-
assert job.name == project.name
220225

221-
url_field_names = {name for name in job.model_fields if name.endswith("url")}
222-
assert all(getattr(job, _) for _ in url_field_names)
226+
non_propagated_fields = {
227+
name for name in job.model_fields if name.endswith("url")
228+
}.union({"name"})
229+
assert all(getattr(job, _) for _ in non_propagated_fields)
223230

224231
# this tends to be a problem
225232
assert job.inputs_checksum == expected_job.inputs_checksum
226-
assert job.model_dump(exclude=url_field_names) == expected_job.model_dump(
227-
exclude=url_field_names
233+
assert job.model_dump(exclude=non_propagated_fields) == expected_job.model_dump(
234+
exclude=non_propagated_fields
228235
)
229236

230237

0 commit comments

Comments
 (0)