This repository was archived by the owner on Jun 10, 2026. It is now read-only.
feat(testing-sdk): add dual-mode integration testing infrastructure#73
Merged
1 commit merged intoOct 30, 2025
Merged
Conversation
vip-amzn
force-pushed
the
integ_tests_new
branch
from
d322404 to
e3782ca
Compare
vip-amzn
changed the title
vip-amzn
force-pushed
the
integ_tests_new
branch
5 times, most recently
from
87f2155 to
dad2b53
Compare
yaythomas
suggested changes
Oct 24, 2025
rarepolz
reviewed
Oct 24, 2025
vip-amzn
force-pushed
the
integ_tests_new
branch
17 times, most recently
from
a1457da to
c5d1d71
Compare
vip-amzn
force-pushed
the
integ_tests_new
branch
20 times, most recently
from
df1b670 to
3aa49a6
Compare
Add comprehensive integration testing infrastructure that supports both local (in-memory) and cloud (AWS Lambda) test execution modes with a unified test interface.
vip-amzn
force-pushed
the
integ_tests_new
branch
from
3aa49a6 to
3a1eb08
Compare
ghost
approved these changes
Oct 30, 2025
Comment on lines
+15
to
+26
| def simulated_get_item(name: str) -> dict[str, Any] | None: | ||
| """Simulate getting an item that may fail randomly.""" | ||
| # Fail 50% of the time | ||
| if random() < 0.5: # noqa: S311 | ||
| msg = "Random failure" | ||
| raise RuntimeError(msg) | ||
|
|
||
| # Simulate finding item after some attempts | ||
| if random() > 0.3: # noqa: S311 | ||
| return {"id": name, "data": "item data"} | ||
|
|
||
| return None |
There was a problem hiding this comment.
If we introduce randomness like this, we should seed the random so that we don't get transient failures.
There was a problem hiding this comment.
So either seeding, or mocking the response and knowing the sequence. I prefer large sequence of randoms tested over multiple seeds.
ghost
merged commit 
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add comprehensive integration testing infrastructure that supports both local (in-memory) and cloud (AWS Lambda) test execution modes with a unified test interface.
Issue #, if available:
Description of changes:
Key Features:
New Infrastructure:
Configuration Updates: