TBOX-185, TBOX-139: Improve operation.wait() function; improve test coverage #175
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
↪️ Pull Request
For TBOX-185, this updates
tamr_toolbox/utils/operation.py
's functionwait
to use the monitor function.It also adds a catch to monitor function to return immediately if the operation is complete (in testing, I observed that if I accidentally sent the operation ID of a complete job to the monitor function, it would poll it indefinitely waiting for a state change).
Along with testing for these changes, this PR adds test coverage for the
operations.py
functions to address TBOX-139. Test coverage foroperations.py
is 100% with these changes.✔️ PR Todo
main()
function in your scripts