health: publish test results to code cov by WilliamBergamin · Pull Request #1677 · slackapi/python-slack-sdk · GitHub
Skip to content

health: publish test results to code cov#1677

Merged
WilliamBergamin merged 2 commits into
mainfrom
code-cov-test-reporting
Mar 28, 2025
Merged

health: publish test results to code cov#1677
WilliamBergamin merged 2 commits into
mainfrom
code-cov-test-reporting

Conversation

@WilliamBergamin

Copy link
Copy Markdown
Contributor

Summary

This PR is inspired by slackapi/node-slack-sdk#2178

It updates the CI pipeline to publish the test results to codcov and provide some neat reporting details

Testing

Category

  • slack_sdk.web.WebClient (sync/async) (Web API client)
  • slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
  • slack_sdk.socket_mode (Socket Mode client)
  • slack_sdk.signature (Request Signature Verifier)
  • slack_sdk.oauth (OAuth Flow Utilities)
  • slack_sdk.models (UI component builders)
  • slack_sdk.scim (SCIM API client)
  • slack_sdk.audit_logs (Audit Logs API client)
  • slack_sdk.rtm_v2 (RTM client)
  • /docs (Documents)
  • /tutorial (PythOnBoardingBot tutorial)
  • tests/integration_tests (Automated tests for this library)

Requirements

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes.

@WilliamBergamin WilliamBergamin added tests M-T: Testing work only python Pull requests that update Python code labels Mar 28, 2025
@WilliamBergamin WilliamBergamin self-assigned this Mar 28, 2025

@mwbrooks mwbrooks left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ Love it!

@codecov

codecov Bot commented Mar 28, 2025

Copy link
Copy Markdown

@zimeg zimeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! And these results are so far more interesting to me than those before. Perhaps a bias of what's recent:

https://app.codecov.io/gh/slackapi/python-slack-sdk/tests/code-cov-test-reporting

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 ✨

It's so great that pytest has built in junit support!

Comment thread .github/workflows/ci-build.yml Outdated
@WilliamBergamin WilliamBergamin merged commit 0b2c91e into main Mar 28, 2025
@WilliamBergamin WilliamBergamin deleted the code-cov-test-reporting branch March 28, 2025 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests that update Python code tests M-T: Testing work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants