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
test: Meta transaction deserialization #58
Conversation
src/s3_fetchers.rs
Outdated
}; | ||
|
||
#[async_trait] | ||
pub trait GetObject { |
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.
Implementation taken from https://docs.aws.amazon.com/sdk-for-rust/latest/dg/testing.html
3c41f86
to
2c69bac
Compare
83629e3
to
24a3cce
Compare
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.
I've committed changes that fix the errors preventing us from moving on.
Left a few comments with my concerns.
4ded912
to
d236ec9
Compare
54e5fab
to
be3aea7
Compare
async-stream = "0.3.3" | ||
|
||
[dev-dependencies] | ||
aws-smithy-http = "0.53.0" |
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.
Needed for constructing an S3 response body via SdkBody
@khorolets this is ready for review now. Thanks a lot for you help! |
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 overall. I've left a few change requests and we're good to go.
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.
Great job!
This PR introduces tests for
s3_fetchers
, with the goal of testing the deserialization of meta transactions. It does this by creating a newtrait
to abstract awayaws_sdk_s3::Client
providing the ability to inject fake implementations which have mock S3 responses.I've also added another step to the existing github action which will run the tests.