Skip to content

test: add unit tests for github#3280

Open
anakin87 wants to merge 1 commit intomainfrom
github-tests
Open

test: add unit tests for github#3280
anakin87 wants to merge 1 commit intomainfrom
github-tests

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented May 6, 2026

Proposed Changes:

  • add more unit tests for GitHub integration, focusing on uncovered code paths (total coverage was below our 90% standard)

How did you test it?

CI

Checklist

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Coverage report (github)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  integrations/github/src/haystack_integrations/components/connectors/github
  file_editor.py
  pr_creator.py
  repo_forker.py
  repo_viewer.py
  integrations/github/src/haystack_integrations/tools/github
  utils.py
Project Total  

This report was generated by python-coverage-comment-action

@anakin87 anakin87 marked this pull request as ready for review May 6, 2026 12:28
@anakin87 anakin87 requested a review from a team as a code owner May 6, 2026 12:28
@anakin87 anakin87 requested review from bogdankostic and removed request for a team May 6, 2026 12:28
@anakin87 anakin87 self-assigned this May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant