Skip to content

Commit 21c656f

Browse files
RobertLuciandeliahu
authored andcommitted
Fix E2E load test (for batch) (#2361)
(cherry picked from commit 6ef5e57)
1 parent 005140d commit 21c656f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test/e2e/e2e/tests.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -796,12 +796,14 @@ def test_load_batch(
796796
printer("checking the jobs' responses")
797797
for job_spec in job_specs:
798798
job_id: str = job_spec["job_id"]
799-
job_status = requests.get(f"{api_endpoint}?jobID={job_id}").json()["job_status"]
799+
job = requests.get(f"{api_endpoint}?jobID={job_id}").json()
800+
job_status = job["job_status"]
801+
job_metrics = job["metrics"]
800802

801803
assert (
802804
job_status["batches_in_queue"] == 0
803805
), f"there are still batches in queue ({job_status['batches_in_queue']}) for job ID {job_id}"
804-
assert job_spec["metrics"]["succeeded"] == math.ceil(items_per_job / batch_size)
806+
assert job_metrics["succeeded"] == math.ceil(items_per_job / batch_size)
805807

806808
num_objects = 0
807809
for page in paginator.paginate(Bucket=bucket, Prefix=os.path.join(key, job_id)):

0 commit comments

Comments
 (0)