Skip to content

Commit 42b7c75

Browse files
committed
test: fixed tests
1 parent 6b8dbc3 commit 42b7c75

File tree

4 files changed

+20
-5
lines changed

4 files changed

+20
-5
lines changed

app/services/processing.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,12 @@ def create_processing_job(
5656
)
5757
record = save_job_to_db(database, record)
5858
return ProcessingJobSummary(
59-
id=record.id, title=record.title, label=request.label, status=record.status
59+
id=record.id,
60+
title=record.title,
61+
label=request.label,
62+
status=record.status,
63+
parameters=request.parameters,
64+
result_link=None,
6065
)
6166

6267

tests/conftest.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@ def fake_processing_job_summary():
5151
title="Test Job",
5252
label=ProcessTypeEnum.OPENEO,
5353
status=ProcessingStatusEnum.CREATED,
54+
parameters={"param1": "value1", "param2": "value2"},
55+
result_link=None,
5456
)
5557

5658

@@ -59,8 +61,6 @@ def fake_processing_job(fake_processing_job_summary, fake_processing_job_request
5961
return ProcessingJob(
6062
**(fake_processing_job_summary.model_dump()),
6163
service=fake_processing_job_request.service,
62-
parameters=fake_processing_job_request.parameters,
63-
result_link="https://foo.bar",
6464
created=datetime.now(),
6565
updated=datetime.now()
6666
)
@@ -71,13 +71,16 @@ def fake_processing_job_record(
7171
fake_processing_job_summary, fake_processing_job_request
7272
):
7373
record = ProcessingJobRecord(
74-
**(fake_processing_job_summary.model_dump()),
74+
id=fake_processing_job_summary.id,
75+
title=fake_processing_job_summary.title,
76+
label=fake_processing_job_summary.label,
77+
status=fake_processing_job_summary.status,
7578
platform_job_id="platform-job-1",
7679
service='{"endpoint":"foo","application":"bar"}',
7780
parameters="{}",
7881
result_link="https://foo.bar",
7982
created=datetime.now(),
80-
updated=datetime.now()
83+
updated=datetime.now(),
8184
)
8285
record.status = ProcessingStatusEnum.CREATED
8386
return record

tests/services/test_processing.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,8 @@ def test_create_processing_job_calls_platform_execute(
6262
title=fake_job.title,
6363
label=ProcessTypeEnum.OPENEO,
6464
status=ProcessingStatusEnum.CREATED,
65+
parameters=fake_job.parameters,
66+
result_link=None,
6567
)
6668
fake_record = ProcessingJobRecord(
6769
id=fake_result,
@@ -111,6 +113,7 @@ def test_get_processing_jobs_with_active_and_inactive_statuses(
111113
label=ProcessTypeEnum.OGC_API_PROCESS,
112114
title="Finished Job",
113115
status=ProcessingStatusEnum.FAILED,
116+
parameters="{}",
114117
service=json.dumps({"foo": "bar"}),
115118
)
116119
mock_get_jobs.return_value = [fake_processing_job_record, inactive_job]
@@ -166,6 +169,7 @@ def test_get_processing_jobs_with_finished_statuses(
166169
title="Finished Job",
167170
status=ProcessingStatusEnum.FINISHED,
168171
service=json.dumps({"foo": "bar"}),
172+
parameters="{}",
169173
)
170174
finished_job_result = ProcessingJobRecord(
171175
id=3,
@@ -174,6 +178,7 @@ def test_get_processing_jobs_with_finished_statuses(
174178
title="Finished Job",
175179
status=ProcessingStatusEnum.FINISHED,
176180
service=json.dumps({"foo": "bar"}),
181+
parameters="{}",
177182
result_link="https://foo.bar",
178183
)
179184
mock_get_jobs.return_value = [finished_job_no_result, finished_job_result]

tests/services/test_upscaling.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ def make_job(status: ProcessingStatusEnum) -> ProcessingJobSummary:
2323
title="job",
2424
status=status,
2525
label=ProcessTypeEnum.OPENEO,
26+
parameters={"param1": "value1", "param2": "value2"},
27+
result_link=None,
2628
)
2729

2830

0 commit comments

Comments
 (0)