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

feat: Speed up automated tests #15

Merged
merged 14 commits into from
Apr 11, 2023
Merged

feat: Speed up automated tests #15

merged 14 commits into from
Apr 11, 2023

Conversation

kgpayne
Copy link
Collaborator

@kgpayne kgpayne commented Feb 7, 2023

No description provided.

@kgpayne
Copy link
Collaborator Author

kgpayne commented Feb 7, 2023

@pnadolny13 as part of speeding up tests, this also implements the tables config option as an array of strings 🙂

@kgpayne
Copy link
Collaborator Author

kgpayne commented Feb 7, 2023

This is probably as fast as we'll get. Takes tests from ~20mins down to ~2m 🚀

@kgpayne
Copy link
Collaborator Author

kgpayne commented Feb 9, 2023

Depends on sdk#1436

pyproject.toml Outdated Show resolved Hide resolved
tap_snowflake/tap.py Outdated Show resolved Hide resolved
Ken Payne and others added 2 commits April 5, 2023 22:59
Co-authored-by: Edgar R. M. <edgarrm358@gmail.com>
@edgarrmondragon edgarrmondragon changed the title feat: speed up automated tests feat: Speed up automated tests Apr 10, 2023
@edgarrmondragon
Copy link
Member

@kgpayne I've pushed changes to use the SDK 0.23.0. Feel free to merge 😁

@kgpayne kgpayne merged commit fdbbd49 into main Apr 11, 2023
@kgpayne kgpayne deleted the kgpayne/speed-up-tests branch April 11, 2023 09:14
@kgpayne kgpayne mentioned this pull request May 16, 2023
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.

2 participants