test(core): add unit tests for content_processor provider

[MeloMei]

Summary

Add unit tests for crewai.core.providers.content_processor, which
previously had zero test coverage.

Changes

New file: lib/crewai/tests/core/test_content_processor.py (25 tests)

Covers:

• NoOpContentProcessor — default no-op behavior, context handling
• ContentProcessorProvider — Protocol runtime checkability
• get_processor / set_processor — ContextVar-based registration
• process_content — convenience function delegation and argument forwarding
• ContextVar isolation — thread isolation and copy_context() semantics

Tests

All 25 tests pass locally:

uv run pytest lib/crewai/tests/core/test_content_processor.py -v
# 25 passed in 10.75s

<!-- This is an auto-generated comment: release notes by coderabbit.ai -->
## Summary by CodeRabbit

* **Tests**
  * Added a new Python test suite validating the content processor provider API, default/no-op behavior, and context forwarding (including `None` handling).
  * Verified returned values propagate correctly.
  * Added isolation coverage to ensure processor state does not leak across separate contexts or threads.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d33caea8-29ac-4ff7-9c44-60f00c4015ff

📥 Commits

Reviewing files that changed from the base of the PR and between 6882686 and f9056f5.

📒 Files selected for processing (1)

• lib/crewai/tests/core/test_content_processor.py

🚧 Files skipped from review as they are similar to previous changes (1)

• lib/crewai/tests/core/test_content_processor.py

---

📝 Walkthrough

Walkthrough

Adds a new test module lib/crewai/tests/core/test_content_processor.py (253 lines) covering NoOpContentProcessor, ContentProcessorProvider protocol conformance, get_processor/set_processor semantics, the process_content convenience function, and isolation guarantees using contextvars and threading.

Changes

Content Processor Test Suite

Layer / File(s)	Summary
Test setup, NoOpContentProcessor, and protocol conformance lib/crewai/tests/core/test_content_processor.py	Imports module symbols, adds an autouse fixture resetting _content_processor to None before/after each test, asserts NoOpContentProcessor returns content unchanged across all input types and ignores context, and verifies ContentProcessorProvider runtime isinstance checks (positive and negative).
get_processor and set_processor API behavior lib/crewai/tests/core/test_content_processor.py	Tests get_processor default stability, override via set_processor, and revert-to-default after clearing. Tests set_processor replacement semantics and the invariant that setting a processor does not mutate the default NoOpContentProcessor instance.
process_content convenience function lib/crewai/tests/core/test_content_processor.py	Verifies process_content dispatches to the active processor, passes context=None by default, forwards an explicit context to the processor, invokes custom processors, and propagates return values.
ContextVar and thread isolation lib/crewai/tests/core/test_content_processor.py	Confirms processors set inside copy_context().run() do not leak to the parent context, processors in separate threads do not cross-contaminate, and a processor set in the main thread is not visible in a newly spawned child thread.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 15.15% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely describes the primary change: adding unit tests for the content_processor provider module in the core package.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Adds initial unit test coverage for crewai.core.providers.content_processor, validating the default no-op processor, Protocol runtime behavior, ContextVar-backed registration, and isolation semantics across copied contexts and threads.

Changes:

• Introduces a new pytest module with focused unit tests for NoOpContentProcessor, get_processor/set_processor, and process_content.
• Verifies ContextVar isolation behavior via contextvars.copy_context() and multi-threaded scenarios.
• Adds assertions around default processor behavior and delegation/argument-forwarding.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@lib/crewai/tests/core/test_content_processor.py`:
- Around line 208-229: The thread synchronization calls barrier.wait() and the
thread join calls (t_a.join() and t_b.join()) lack timeout parameters, which can
cause the test to hang indefinitely if a thread fails or doesn't exit. Add a
timeout parameter (e.g., timeout=5 or a similar reasonable value in seconds) to
both barrier.wait() calls in the thread_a and thread_b functions, and to both
t_a.join() and t_b.join() calls. Apply the same timeout fixes to the similar
test pattern mentioned at lines 246-249 to ensure all thread synchronization
points have bounded waits.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7efeeb7f-a6aa-4ca6-be84-d61ca8f2006c

📥 Commits

Reviewing files that changed from the base of the PR and between 218dc82 and 6882686.

📒 Files selected for processing (1)

• lib/crewai/tests/core/test_content_processor.py