-
Notifications
You must be signed in to change notification settings - Fork 90
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
feat: end-to-end tests for core modules and idempotency #970
feat: end-to-end tests for core modules and idempotency #970
Conversation
14d5059
to
94252a4
Compare
For some reason I don't explain, I cannot set the GitHub Action for e2e tests. I pause on this, already spent too much time. This is what I used:
And the error : aws-actions/configure-aws-credentials#460. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeromevdl some quick initial feedback to get the ball rolling. I need to dive a bit deeper into this still.
powertools-e2e-tests/src/test/java/software/amazon/lambda/powertools/IdempotencyE2ET.java
Show resolved
Hide resolved
...ools-e2e-tests/src/test/java/software/amazon/lambda/powertools/testutils/Infrastructure.java
Show resolved
Hide resolved
private Stack createStackWithLambda() { | ||
Stack stack = new Stack(app, stackName); | ||
List<String> packagingInstruction = Arrays.asList( | ||
"/bin/sh", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the powertools test suite otherwise run on windows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question... I don't know!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it does - tested quickly on windows - do we want to maintain that?
powertools-e2e-tests/src/test/java/software/amazon/lambda/powertools/testutils/JavaRuntime.java
Show resolved
Hide resolved
...-tests/src/test/java/software/amazon/lambda/powertools/testutils/metrics/MetricsFetcher.java
Show resolved
Hide resolved
...tests/src/test/java/software/amazon/lambda/powertools/testutils/tracing/SegmentDocument.java
Show resolved
Hide resolved
...src/main/java/software/amazon/lambda/powertools/idempotency/internal/IdempotencyHandler.java
Outdated
Show resolved
Hide resolved
Hey @jeromevdl i've gone through the whole PR now and marked up some bits. Let me know if I there's anything else I can do to help. |
- rename to avoid execution during test phase - use a profile to execute them on demand only
94252a4
to
7b81e22
Compare
We have our answers, obviously it cannot work on java 8... 🤬 Will have to make it compatible 😫 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@jeromevdl the one question I have is - did you get this running in the github builds in the end? |
@scottgerring Nope, was not working... I don't know why, if you want to have a look. |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Description of changes:
Adding end-to-end tests (happy path) for core modules (logging, metrics, tracing) and idempotency. Leverage AWS CDK + S3 and CloudFormation SDK to deploy a Lambda function and the appropriate infrastructure (ex: DDB table for idempotency).
Checklist
Breaking change checklist
RFC issue #:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.