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
DynamoDB: Use fixtures to create and clean up test tables #8747
Conversation
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 + small nit
tests/integration/test_dynamodb.py
Outdated
""" | ||
This fixture returns a DynamoDB table for testing. | ||
""" | ||
table_name = "ddb-test-table" |
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.
table_name = "ddb-test-table" | |
table_name = "ddb-test-table" |
Would suggest making this unique to avoid any potential issues/conflicts and allow for potential parallelization in the future
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.
Fixed in e274007
This PR makes a small improvement to the DynamoDB test suite.
Test table names that were earlier decalared as constants are now converted to fixtures which do proper clean up after tests finish.