Skip to content

chore(deleter): add GetBalance and subscription List to deleter mocks - #1865

Open
whoAbhishekSah wants to merge 1 commit into
org-delete-ack-protofrom
org-delete-preflight-mocks
Open

chore(deleter): add GetBalance and subscription List to deleter mocks#1865
whoAbhishekSah wants to merge 1 commit into
org-delete-ack-protofrom
org-delete-preflight-mocks

Conversation

@whoAbhishekSah

Copy link
Copy Markdown
Member

Part of #1837. Middle of a 3-PR stack: proto → #this → pre-flight code (#1857).

Regenerated mocks for the deleter's SubscriptionService and CreditService interfaces, which gain List and GetBalance in the next PR. The org delete pre-flight check needs them to read subscription states and token balances before deleting anything.

Only generated code here. The extra mock methods are harmless until the interfaces change: the CascadeDeleter mock is not in this PR because its DeleteOrganization signature change must land together with the interface and handler to keep every layer compiling.

🤖 Generated with Claude Code

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview Aug 10, 2026 6:28am

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f2d18dc4-6a29-49d5-a71f-52e055236220

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 31357002580

Warning

No base build found for commit bbf43fc on org-delete-ack-proto.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 47.704%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 39443
Covered Lines: 18816
Line Coverage: 47.7%
Coverage Strength: 15.42 hits per line

💛 - Coveralls

Generated for the org delete pre-flight check, which needs to read
subscription states and token balances before deleting anything. The
interface additions land in the next change; extra mock methods are
harmless until then.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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