Skip to content

Commit

Permalink
Fix Modal unit tests following change in #382 that added a 'name' par…
Browse files Browse the repository at this point in the history
…ameter
  • Loading branch information
tomwhite committed Feb 19, 2024
1 parent 4ff1cfa commit d5f5589
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions cubed/tests/runtime/test_modal_async.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,20 +35,22 @@
@stub.function(
image=image, secret=modal.Secret.from_name("my-aws-secret"), retries=2, timeout=10
)
def deterministic_failure_modal(i, path=None, timing_map=None):
return deterministic_failure(path, timing_map, i)
def deterministic_failure_modal(i, path=None, timing_map=None, *, name=None):
return deterministic_failure(path, timing_map, i, name=name)


@stub.function(image=image, secret=modal.Secret.from_name("my-aws-secret"), timeout=10)
def deterministic_failure_modal_no_retries(i, path=None, timing_map=None):
return deterministic_failure(path, timing_map, i)
def deterministic_failure_modal_no_retries(i, path=None, timing_map=None, *, name=None):
return deterministic_failure(path, timing_map, i, name=name)


@stub.function(
image=image, secret=modal.Secret.from_name("my-aws-secret"), retries=2, timeout=300
)
def deterministic_failure_modal_long_timeout(i, path=None, timing_map=None):
return deterministic_failure(path, timing_map, i)
def deterministic_failure_modal_long_timeout(
i, path=None, timing_map=None, *, name=None
):
return deterministic_failure(path, timing_map, i, name=name)


async def run_test(app_function, input, use_backups=False, batch_size=None, **kwargs):
Expand Down

0 comments on commit d5f5589

Please sign in to comment.