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-24569] New Stream inclusion: subtasks #56
Conversation
"format": "date-time" | ||
} | ||
} | ||
} |
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.
Add blank/empty line at the end of file
return subtasks + subtasks_children | ||
|
||
|
||
Context.stream_objects["subtasks"] = SubTasks |
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.
Add the empty line at the end of the file and wherever it is missing
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.
i am trying to keep it consistent. since all the other stream files don't have a new line at EOF, i will skip these formatting changes.
# running sync with multiple date timestamps accross different streams due to differences in bookmark values | ||
self.run_test("2021-11-09T00:00:00Z", "2023-11-10T00:00:00Z", {"projects",}) | ||
self.run_test("2023-11-28T00:00:00Z", "2023-11-30T00:00:00Z", {"subtasks",}) | ||
self.run_test("2019-01-28T00:00:00Z", "2023-11-30T00:00:00Z", self.expected_streams() - {"subtasks","projects"}) |
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.
Any reason, third execution doesn't have subtasks and projects streams?
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.
all streams do not contain data in a singular date range, hence to validate bookmarking for all streams the tests are being run with different date ranges for these two specific streams
Description of change
subtasks
streamManual QA steps
Risks
Rollback steps