Skip to content

Commit

Permalink
Enable two more e2e tests
Browse files Browse the repository at this point in the history
This commit enables two additional e2e tests.
- UserErrors
- CancelInvocation
  • Loading branch information
igalshilman committed Aug 7, 2024
1 parent e544a2c commit 19222bb
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 10 deletions.
6 changes: 0 additions & 6 deletions test-services/exclusions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,12 @@ exclusions:
"alwaysSuspending":
- "dev.restate.sdktesting.tests.AwaitTimeout"
- "dev.restate.sdktesting.tests.State"
- "dev.restate.sdktesting.tests.UserErrors"
"default":
- "dev.restate.sdktesting.tests.AwaitTimeout"
- "dev.restate.sdktesting.tests.CancelInvocation"
- "dev.restate.sdktesting.tests.KafkaIngress"
- "dev.restate.sdktesting.tests.State"
- "dev.restate.sdktesting.tests.UserErrors"
"lazyState":
- "dev.restate.sdktesting.tests.State"
"singleThreadSinglePartition":
- "dev.restate.sdktesting.tests.AwaitTimeout"
- "dev.restate.sdktesting.tests.CancelInvocation"
- "dev.restate.sdktesting.tests.KafkaIngress"
- "dev.restate.sdktesting.tests.State"
- "dev.restate.sdktesting.tests.UserErrors"
6 changes: 3 additions & 3 deletions test-services/services/cancel_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ async def verify_test(ctx: ObjectContext) -> bool:
return False
return state


blocking_service = VirtualObject("CancelTestBlockingService")

@blocking_service.handler()
Expand All @@ -58,6 +58,6 @@ async def block(ctx: ObjectContext, op: BlockingOperation):
name, uncompleteable = ctx.awakeable()
await uncompleteable

@blocking_service.handler(name="isUnblocked")
async def is_unblocked(ctx: ObjectContext):
@blocking_service.handler(name="isUnlocked")
async def is_unlocked(ctx: ObjectContext):
return None
2 changes: 1 addition & 1 deletion test-services/services/failing.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ def side_effect():
async def terminally_failing_side_effect(ctx: ObjectContext):

def side_effect():
raise TerminalError(message="Terminally failing side effect")
raise TerminalError(message="failed side effect")

await ctx.run("sideEffect", side_effect)
raise ValueError("Should not reach here")

0 comments on commit 19222bb

Please sign in to comment.