Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: TapDiscoveryTest remove catalog if one is passed #1364

Merged
merged 4 commits into from
Feb 1, 2023

Conversation

kgpayne
Copy link
Contributor

@kgpayne kgpayne commented Jan 30, 2023

@codecov
Copy link

codecov bot commented Jan 30, 2023

Codecov Report

Merging #1364 (f61e608) into main (53e848b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1364   +/-   ##
=======================================
  Coverage   85.19%   85.19%           
=======================================
  Files          54       54           
  Lines        4693     4694    +1     
  Branches      797      798    +1     
=======================================
+ Hits         3998     3999    +1     
  Misses        504      504           
  Partials      191      191           
Impacted Files Coverage Δ
singer_sdk/testing/tap_tests.py 81.57% <100.00%> (+0.16%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kgpayne
Copy link
Contributor Author

kgpayne commented Jan 31, 2023

@edgarrmondragon if you have a chance to look at this small change today I would hugely appreciate it 🙏 It would allow me to use automated tests, and the framework, for testing the changes in tap-snowflake#13 by unblocking tap-snowflake#14 🙂

@kgpayne kgpayne merged commit ed08850 into main Feb 1, 2023
@kgpayne kgpayne deleted the kgpayne/fix-test-tap-discovery branch February 1, 2023 11:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants