Title: Test Coverage: End to End Tests for Dataflow Runner · Issue #595 · feast-dev/feast · GitHub
Open Graph Title: Test Coverage: End to End Tests for Dataflow Runner · Issue #595 · feast-dev/feast
X Title: Test Coverage: End to End Tests for Dataflow Runner · Issue #595 · feast-dev/feast
Description: Is your feature request related to a problem? Please describe. Currently, Feast's end to end tests only verify correctness with Feast Core's Direct Runner, leaving a gap in test coverage for the Dataflow runner. This is an issue a we are...
Open Graph Description: Is your feature request related to a problem? Please describe. Currently, Feast's end to end tests only verify correctness with Feast Core's Direct Runner, leaving a gap in test coverage for the Da...
X Description: Is your feature request related to a problem? Please describe. Currently, Feast's end to end tests only verify correctness with Feast Core's Direct Runner, leaving a gap in test coverage fo...
Opengraph URL: https://github.com/feast-dev/feast/issues/595
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Test Coverage: End to End Tests for Dataflow Runner","articleBody":"**Is your feature request related to a problem? Please describe.**\r\n\r\nCurrently, Feast's end to end tests only verify correctness with Feast Core's [Direct Runner](https://github.com/gojek/feast/blob/2fa30ac1883ad6fdc42168487a0fbafd38020b16/infra/scripts/test-end-to-end.sh#L103), leaving a gap in test coverage for the Dataflow runner. This is an issue a we are unable to verify that new additions to the codebase work well with the Dataflow runner, requiring PR authors to do error prone manual testing.\r\n\r\nCase study: PR #575 \r\n- I submitted small PR #575 which updated how `Runner`s are rendered.\r\n- Since the PR is relatively small and all the tests pass, I assumed that the PR did not introduce any bugs and didn't do any independent testing on my own.\r\n- I only discovered that the PR broke the Dataflow runner after it was merged, leaving a known bug on the master branch.\r\n\r\nThis bug in the PR would have been caught by Feast's test suite, had it tested with the Dataflow runner in addition to the Direct runner.\r\n\r\n**Describe the solution you'd like**\r\nAdd End to End Tests using the Dataflow Runner, in additional to the Direct Runner.","author":{"url":"https://github.com/mrzzy","@type":"Person","name":"mrzzy"},"datePublished":"2020-04-01T16:12:00.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/595/feast/issues/595"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:b3ab8d04-d677-4e83-e828-43ce95230578 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | A0D8:3F8B4C:3A40944:51B5CB3:69792695 |
| html-safe-nonce | 730e364a2921f1483e068fe3793e959fa4293b26e0588d5c4503beab5e82b670 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBMEQ4OjNGOEI0QzozQTQwOTQ0OjUxQjVDQjM6Njk3OTI2OTUiLCJ2aXNpdG9yX2lkIjoiMjQzMDYwMTU2MzE0MDA3MzEwOSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 1d19b8803ddadbddab0c262f2f23d769d15d610ad27840655af44ca2c1faa96b |
| hovercard-subject-tag | issue:592035395 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/feast-dev/feast/595/issue_layout |
| twitter:image | https://opengraph.githubassets.com/805b62fe4aac70ed514da8eff6328d7280d64ad2e722d828b54ef9608f9e5ea2/feast-dev/feast/issues/595 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/805b62fe4aac70ed514da8eff6328d7280d64ad2e722d828b54ef9608f9e5ea2/feast-dev/feast/issues/595 |
| og:image:alt | Is your feature request related to a problem? Please describe. Currently, Feast's end to end tests only verify correctness with Feast Core's Direct Runner, leaving a gap in test coverage for the Da... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | mrzzy |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5c80adc901e87b5b58bfa7505fd128e16f3c9068fea81f6683cdf63c7331a9af |
| turbo-cache-control | no-preview |
| go-import | github.com/feast-dev/feast git https://github.com/feast-dev/feast.git |
| octolytics-dimension-user_id | 57027613 |
| octolytics-dimension-user_login | feast-dev |
| octolytics-dimension-repository_id | 161133770 |
| octolytics-dimension-repository_nwo | feast-dev/feast |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 161133770 |
| octolytics-dimension-repository_network_root_nwo | feast-dev/feast |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 9dcfea5b525a9115ea36bbcd305680ba9ec70338 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width