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
TDL-5784 Add export_events param #56
Conversation
@@ -152,3 +152,4 @@ def test_run(self): | |||
# Pagination test for EU residency server | |||
self.eu_residency = True | |||
self.pagination_test_run() | |||
|
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.
it doesn't look like export events were actually added to this test?
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.
Yeah, created env variable for export_events param and also adjusted start date since data was missing for export stream. So, I had to generate test data for export stream and accordingly adjusted the start date
Description of change
export_events
param to fetch specific events.Manual QA steps
export_events
param value and verified that tap fetch only specified events.0
records when user sets an invalid event name in the config.Risks
Rollback steps