Skip to content

[async worker] run unit tests in CI - #467

Merged
julietshen merged 1 commit into
mainfrom
ayu/test-async-worker-ci
Aug 25, 2026
Merged

[async worker] run unit tests in CI#467
julietshen merged 1 commit into
mainfrom
ayu/test-async-worker-ci

Conversation

@ayubun

@ayubun ayubun commented Aug 25, 2026

Copy link
Copy Markdown
Member

🤖 summary

runs the experimental async worker's existing unit suite in the python quality job so those tests gate every pull request

related issues/tasks

changes made (˶ᵔ ᵕ ᵔ˶)

  • add a focused async worker pytest step after dependency and static checks
  • keep the stable worker testpaths and integration image unchanged

models used

  • anthropic/claude-opus-4-8: ~55%
  • openai/gpt-5.6-sol: ~25%
  • anthropic/claude-fable-5: ~10%
  • anthropic/claude-haiku-4-5: ~10%

add a focused pytest step to the python-quality job that runs
uv run pytest -q osprey_async_worker after dependency install. the
experimental async worker is excluded from root testpaths and the
docker integration image, so its 159 unit tests otherwise never run
in CI
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 59 minutes.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b7b29fcc-faf7-4e42-a608-bbe099bbb0cf

📥 Commits

Reviewing files that changed from the base of the PR and between 1cbf04b and d87e500.

📒 Files selected for processing (1)
  • .github/workflows/code-quality.yml

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ayubun
ayubun marked this pull request as ready for review August 25, 2026 01:26
@ayubun
ayubun requested review from a team, EXBreder, haileyok and vinaysrao1 as code owners August 25, 2026 01:26

@julietshen julietshen 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

@julietshen
julietshen merged commit 2fece68 into main Aug 25, 2026
17 checks passed
@julietshen
julietshen deleted the ayu/test-async-worker-ci branch August 25, 2026 15:12
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.

2 participants