-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Show integration test results on the Action Summary #13072
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
Conversation
@@ -159,7 +159,7 @@ jobs: | |||
body: | | |||
> :white_check_mark: ${{github.event.inputs.connector}} https://github.com/${{github.repository}}/actions/runs/${{github.run_id}} | |||
${{env.PYTHON_UNITTEST_COVERAGE_REPORT}} | |||
> ${{env.PYTHON_SHORT_TEST_SUMMARY_INFO}} | |||
> ${{env.TEST_SUMMARY_INFO}} |
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.
Changed because we are using it for both languages now
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.
this is great! good call
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.
Very cool! I'll try to find time to work something out for the platform builds.
/test source-airtable
|
/test connector=connectors/source-airtable |
/test connector=connectors/destination-snowflake |
/test connector=connectors/source-amazon-ads |
/test connector=connectors/destination-bigquery |
What
Bubbling up errors on our integration test actions so we don't have to look through the logs.
Even just getting the name of the connector at the top level seems helpful.
Also making it so that we can report short summaries on Java connectors. Was previously only Python.
Examples:
\n
)\n
)How
Used
$GITHUB_STEP_SUMMARY
as described here.