-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[BEAM-14532] Add integration testing to fhirio Read transform #17803
[BEAM-14532] Add integration testing to fhirio Read transform #17803
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
The 3 test files I added in |
R: @msbukal |
R: @youngoli |
Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control |
Codecov Report
@@ Coverage Diff @@
## master #17803 +/- ##
=======================================
Coverage 74.02% 74.02%
=======================================
Files 698 698
Lines 92203 92203
=======================================
Hits 68256 68256
Misses 22696 22696
Partials 1251 1251
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM beyond style nits.
R: @youngoli |
Run XVR_GoUsingJava_Dataflow PostCommit |
Run Go Postcommit |
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.
Looks good to me.
…#17803) * add integration tests * adjust import order to follow convention * fixes to follow beam integration testing convention * remove unused function and rename back to fakes_test.go * import needed flags and read them properly * unnest code in test * add filters for specific runners * add missing license * rename member variable and use constant * use better test files * use gcpopts instead of pipeline options * move testing logic to _test file * filter direct runner as well since it doesnt provide the flags needed
Follow up to apache#17748.
Adds integration tests to the Read transform in
fhirio
package.Fixes #21679
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.CHANGES.md
with noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI.