Skip to content

Conversation

raunakab
Copy link
Contributor

Description

Fix failing IMAP tests. No linear ticket assigned to this PR.

How Has This Been Tested?

Tests are now passing.

@raunakab raunakab requested a review from a team as a code owner July 14, 2025 16:49
Copy link

vercel bot commented Jul 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
internal-search ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2025 4:53pm

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Summary

This PR fixes failing IMAP connector tests by updating test data to match the actual email recipient lists. Specifically, 'raunak@onyx.app' has been added as an additional recipient in two test email documents. This is a test-only change that aligns the test expectations with the actual email data being tested against, rather than a change in functionality.

This change fits into the codebase's connector testing framework, which is part of the automated testing pipeline that runs daily via GitHub Actions (as seen in pr-python-connector-tests.yml). IMAP tests are one of many connector tests that verify email integration functionality.

Confidence score: 5/5

  1. This PR is very safe to merge as it only modifies test data, not production code
  2. High confidence score given because the change is minimal, well-scoped, and only affects test assertions
  3. Files needing attention: None - the changes are straightforward and only affect test data

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant