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

Document how to unit test idempotency #631

Closed
heitorlessa opened this issue Aug 21, 2021 · 3 comments
Closed

Document how to unit test idempotency #631

heitorlessa opened this issue Aug 21, 2021 · 3 comments
Assignees
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers help wanted Could use a second pair of eyes/hands p1

Comments

@heitorlessa
Copy link
Contributor

What were you initially searching for in the docs?

How to unit test code when using idempotency utility

Is this related to an existing part of the documentation? Please share a link

Describe how we could make it clearer

Like other utilities, create a section Testing your code with an example test.

If you have a proposed update, please share it here

@heitorlessa heitorlessa added documentation Improvements or additions to documentation good first issue Good for newcomers help wanted Could use a second pair of eyes/hands labels Aug 21, 2021
@lucasam
Copy link

lucasam commented Sep 10, 2021

We are facing issues with UT as well.

Our main issue is that, as DynamoDBPersistenceLayer is eager to create table. Every python script that we use this annotation needs have mocks in place in advance. Even to test non annotated methods at the same python file.

My question is:

  • Am i missing something? (This is why i am interacting with this ticket)
  • Should i cut a ticket and raise a PR to make DynamoDBPersistenceLayer to lazy create boto resources?

@heitorlessa
Copy link
Contributor Author

Hey @lucasam if you could contribute that would be awesome, as I'm on vacation for two more weeks.

@heitorlessa
Copy link
Contributor Author

Thanks a lot @cakepietoast !!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers help wanted Could use a second pair of eyes/hands p1
Projects
None yet
Development

No branches or pull requests

3 participants