Skip to content

chore(test): move invoke/start-api/start-lambda durable function test to unit test#8635

Merged
roger-zhangg merged 1 commit intodevelopfrom
DAR_test
Feb 6, 2026
Merged

chore(test): move invoke/start-api/start-lambda durable function test to unit test#8635
roger-zhangg merged 1 commit intodevelopfrom
DAR_test

Conversation

@roger-zhangg
Copy link
Member

@roger-zhangg roger-zhangg commented Feb 6, 2026

Which issue(s) does this change fix?

Why is this change necessary?

There's no Durable function test coverage in unit test, causing
#8608 #8590 unknowingly broke Durable function.

How does it address the issue?

Move few DAR test from integ test to unit test.

What side effects does this change have?

N/A as we still have other DAR test coverage in integ test

Mandatory Checklist

PRs will only be reviewed after checklist is complete

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@roger-zhangg roger-zhangg requested a review from a team as a code owner February 6, 2026 20:38
@roger-zhangg
Copy link
Member Author

Integ test: https://github.com/aws/aws-sam-cli/actions/runs/21765301977
DAR test failure is expected. We have another incomming PR to fix it

@roger-zhangg roger-zhangg changed the title chore(test): add dar to unit test chore(test): move invoke/start-api/start-lambda durable function test to unit test Feb 6, 2026
Copy link
Member

@bchampp bchampp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@roger-zhangg roger-zhangg merged commit dea7e5c into develop Feb 6, 2026
68 of 75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants