Skip to content

docs(readme): reconcile test count claims — ~6,300 across all packages - #1860

Merged
LucasSantana-Dev merged 5 commits into
mainfrom
fix/1820-test-count-docs
Jul 22, 2026
Merged

LucasSantana-Dev merged 5 commits into
mainfrom
fix/1820-test-count-docs

Conversation

@LucasSantana-Dev

@LucasSantana-Dev LucasSantana-Dev commented Jul 20, 2026 •

Copy link
Copy Markdown
Owner

Fixes #1820

Problem

README claimed ~2,500 tests while the social-preview image claimed 849+. Both were wrong/inconsistent.

Actual counts

Package Tests
backend 1,209
bot 2,554 (+ 95 it.each)
shared 1,377
frontend 1,190
Total ~6,300

Changes

  • Updated all README references from ~2,500 to ~6,300

Follow-up

The social-preview image (assets/lucky-social-preview.png) still shows '849+ Tests' and needs regeneration via the brand asset tool.


Summary by cubic

Updated README test count to ~6,300 across all packages and aligned CI/CD references. Fixes #1820.

  • Bug Fixes
    • Replaced all "~2500 tests" mentions in README with "~6,300 tests" (intro, CI, stack table, scripts, contributing).
    • Social preview image still shows "849+ Tests" — regenerate via the brand asset tool.

Written for commit f2b64d2. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Documentation
    • Updated testing documentation to reflect the current suite size of approximately 6,300 tests.
    • Revised testing references across feature badges, CI/CD details, development instructions, and contribution guidelines.

Fixes #1820

Actual counts:
- backend: 1,209 tests
- bot: 2,554 tests (+ 95 it.each)
- shared: 1,377 tests
- frontend: 1,190 tests

Total: ~6,300 tests (was ~2,500, which reflected bot-package only).
@coderabbitai

coderabbitai Bot commented Jul 20, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

README test-count references were updated from approximately 2,500 to 6,300 tests across badges, project descriptions, development instructions, and contributor verification guidance.

Changes

README Documentation Update

Layer / File(s) Summary
Update documented test counts
README.md
Badges, CI/CD and technology descriptions, test-running instructions, and the contributor verification checklist now reference approximately 6,300 tests.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the README update to align the documented test count with ~6,300 across packages.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/1820-test-count-docs

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 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 `@README.md`:
- Line 10: Regenerate the README social-preview image so its displayed test
count matches the current “~6,300 tests” claim instead of “849+ Tests”. Update
the referenced preview asset while preserving the existing README metadata and
layout.
- Line 10: Reconcile the aggregate test-count references in the README,
including the headline and the other repeated count entries, with the supplied
package totals. Update them consistently to approximately 6,400/6,425 tests, or
explicitly document which tests are excluded if retaining the current count.
🪄 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c330375a-2f7f-4f78-bb43-7cb669784086

📥 Commits

Reviewing files that changed from the base of the PR and between c658948 and 981fe71.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Auto-approved: Updates stale test count in README to match current monorepo totals. Purely cosmetic documentation fix with no code or behavior changes.

Re-trigger cubic

@github-actions

Copy link
Copy Markdown

Failed to generate code suggestions for PR

@LucasSantana-Dev

Copy link
Copy Markdown
Owner Author

Kimi review (kimi-code/kimi-for-coding, via local subscription)

• VERDICT: CLEAN
README test-count updates only; no code or security concerns.


Head 600c95e. Posted by kimi-review-watch (launchd).

@LucasSantana-Dev

Copy link
Copy Markdown
Owner Author

Merge note (admin): required checks all pass; thread (social-preview regeneration) resolved as advisory follow-up. BLOCKED is the CodeRabbit rate-limit rollup artifact.

@sonarqubecloud

Copy link
Copy Markdown

@LucasSantana-Dev
LucasSantana-Dev merged commit 3263431 into main Jul 22, 2026
38 checks passed
@LucasSantana-Dev
LucasSantana-Dev deleted the fix/1820-test-count-docs branch July 22, 2026 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: conflicting test-count claims — README "~2500" vs social-preview "849+"

1 participant