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

Refactor AWS tests to use new LocalStack fixture #195

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gregfurman
Copy link
Collaborator

Changes

  • Create new LocalStack fixture GetLocalStack for AWS tests
  • Support using LS pro image when testing
  • Use new fixture in tests
  • Refactor ElasticSearch ouput to more simply sign requests when sending to AWS
  • Refactor ElasticSearch test to correct use LS

Note: This approach will need a re-work if we want to include Snowflake testing with LS (available in the pro version which the CI has access to).

@gregfurman gregfurman self-assigned this Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant