Skip to content
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

[request]: Support serialization / deserialization of data structures (regression testing) #211

Open
jasonmccampbell opened this issue Aug 29, 2021 · 1 comment
Labels
feature-request A feature should be added or improved. p2 This is a standard priority issue

Comments

@jasonmccampbell
Copy link

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue, please leave a comment

Tell us about your request
Support serialization and deserialization of Textract and other data structures returned as a result of queries to AWS services

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
For regression testing, I find it useful to cache Textract results to make sure they are being processed correctly for known documents. For this use case, it is useful to save the returned JSON and be able to deserialize it to the same data structures
which are returned from live calls.

Are you currently working around this issue?
I currently do this in Rusoto, it would be convenient to be able to deserialize saved JSON back to a GetDocumentAnalysisOutput structure. It looks like the existing serialization code is private and designed to work with HTTP responses.

If there is already an existing mechanism for this, please let me know.

Additional context
The library works really well - thank you!

Attachments
None

@jasonmccampbell jasonmccampbell added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Aug 29, 2021
@rcoh rcoh added needs-triage This issue or PR still needs to be triaged. and removed needs-triage This issue or PR still needs to be triaged. labels Sep 13, 2021
@rcoh
Copy link
Contributor

rcoh commented Oct 21, 2021

Related: #199

Thanks for the request! We're going to investigate ways to enable different sorts of record/replay as well as mock client construction to address these use cases.

@rcoh rcoh removed the needs-triage This issue or PR still needs to be triaged. label Oct 21, 2021
@rcoh rcoh changed the title [request]: Support serialization / deserialization of data structures [request]: Support serialization / deserialization of data structures (regression testing) Oct 21, 2021
@jmklix jmklix added the p2 This is a standard priority issue label Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

3 participants