Skip to content

Add scenario tests with Fundamental level for Stepfunctions#7793

Merged
dominikschubert merged 4 commits intomasterfrom
test_sfn-v2-scenarios
Mar 8, 2023
Merged

Add scenario tests with Fundamental level for Stepfunctions#7793
dominikschubert merged 4 commits intomasterfrom
test_sfn-v2-scenarios

Conversation

@dominikschubert
Copy link
Member

@dominikschubert dominikschubert commented Mar 3, 2023

Adds tests based on workflow samples from serverlessland.com

  • path-based-on-data
  • wait-for-callback
  • step-functions-calling-api-gateway

I've had to drop a lot of the samples since they were a poor fit (either too complicated or too simple and some of them didn't even properly allow testing all paths defined in the workflow definition). The rest were all simplified/fixed.

All tests are designed to be able to take a single CloudFormation template as an input with lambda code & workflow definitions being included in-line in the template.

@dominikschubert dominikschubert self-assigned this Mar 3, 2023
@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests March 3, 2023 09:32 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

LocalStack integration with Pro

       3 files  ±0         3 suites  ±0   1h 38m 46s ⏱️ + 5m 34s
1 782 tests +3  1 403 ✔️ +3  379 💤 ±0  0 ±0 
2 508 runs  +3  1 779 ✔️ +3  729 💤 ±0  0 ±0 

Results for commit 0c1a2d8. ± Comparison against base commit 83d7780.

♻️ This comment has been updated with latest results.

@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests March 3, 2023 14:24 — with GitHub Actions Inactive
@dominikschubert dominikschubert requested a review from MEPalma March 3, 2023 14:27
@dominikschubert dominikschubert marked this pull request as ready for review March 3, 2023 14:35
@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests March 3, 2023 14:35 — with GitHub Actions Inactive
@dominikschubert dominikschubert force-pushed the test_sfn-v2-scenarios branch from e0e99a9 to ceb8788 Compare March 4, 2023 09:38
@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests March 4, 2023 09:38 — with GitHub Actions Inactive
@dominikschubert dominikschubert force-pushed the test_sfn-v2-scenarios branch from ceb8788 to 0c1a2d8 Compare March 6, 2023 08:03
@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests March 6, 2023 08:03 — with GitHub Actions Inactive
@coveralls
Copy link

Coverage Status

Coverage: 85.093% (+0.01%) from 85.078% when pulling 0c1a2d8 on test_sfn-v2-scenarios into 83d7780 on master.

@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests March 8, 2023 14:29 — with GitHub Actions Inactive
@dominikschubert dominikschubert merged commit 3224002 into master Mar 8, 2023
@dominikschubert dominikschubert deleted the test_sfn-v2-scenarios branch March 8, 2023 17:00
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.

4 participants