Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update e2e tests #4

Closed
wants to merge 2 commits into from
Closed

Update e2e tests #4

wants to merge 2 commits into from

Conversation

kaibocai
Copy link
Member

@kaibocai kaibocai commented Dec 6, 2023

This PR

  1. fixes issues in current e2e tests
  • removed incorrect assert, for ex
    expect(state?.serializedInput).toBeUndefined();
    expect(state?.serializedOutput).toBeUndefined();
    expect(state?.serializedCustomStatus).toBeUndefined();

serializedInput, serializedOutput and serializedCustomStatus will be '' instead of undefined

  1. add more tests for new scenarios.
  • single timer
  • Fan-in/Fan-out
  1. improve current tests for better readablity
  • continue as new
  • sub-orchestration

Currently two test cases are failing, both time out.

it("should be able to use the sub-orchestration for fan-out", async () => {

it("should allow to continue as new", async () => {

@kaibocai
Copy link
Member Author

kaibocai commented Dec 7, 2023

combined into PR #14

@kaibocai kaibocai closed this Dec 7, 2023
@kaibocai kaibocai deleted the kaibocai/improve-e2etests branch December 11, 2023 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant