Skip to content

Conversation

@6ba3i
Copy link
Contributor

@6ba3i 6ba3i commented Feb 11, 2026

What problem does this PR solve?

This PR reduces missing coverage in the ragflow codebase by adding a small set of behavior driven tests that exercise real API and SDK contracts. It targets previously uncovered branches in HTTP API, Web API, SDK modules, and unit level auth/app helpers, focusing on request validation, error shaping, and deterministic success paths. it also adds a new testcase (test_unit) in the CI

Type of change

  • Refactoring
  • Other (please describe): Test coverage improvement (adds new tests and extends existing ones)

@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. 🧪 test Pull requests that update test cases. labels Feb 11, 2026
@6ba3i 6ba3i added the ci Continue Integration label Feb 11, 2026
@6ba3i 6ba3i marked this pull request as draft February 11, 2026 07:58
@6ba3i 6ba3i marked this pull request as ready for review February 11, 2026 07:58
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.70%. Comparing base (88920d2) to head (0c01e2a).
⚠️ Report is 11 commits behind head on main.

Files with missing lines Patch % Lines
api/apps/document_app.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13110      +/-   ##
==========================================
+ Coverage   44.05%   48.70%   +4.64%     
==========================================
  Files          43       43              
  Lines        9415     9506      +91     
  Branches      107      107              
==========================================
+ Hits         4148     4630     +482     
+ Misses       5248     4843     -405     
- Partials       19       33      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@6ba3i 6ba3i marked this pull request as draft February 11, 2026 09:22
@6ba3i 6ba3i marked this pull request as ready for review February 12, 2026 07:46
@6ba3i 6ba3i marked this pull request as draft February 12, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continue Integration size:XXL This PR changes 1000+ lines, ignoring generated files. 🧪 test Pull requests that update test cases.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant