yml2213andClaude Opus 4.8
e8621728fd
钱包充值为开发态测试功能(生产环境本就禁用),且支付回调存在重复入账风险:入账与标记 paid 两步非原子、wallet_ledger 去重无唯一索引、幂等键使用了可变的 ProviderOrderID。直接删除该功能从根本上消除风险。 后端: - payment 移除 StartWalletRecharge/QueryWalletRecharge 及相关 handler/DTO/常量;confirmPaid 增加 OrderID 守卫;解除对 wallet 仓库的依赖 - wallet 移除 Recharge/ConfirmRechargeFromChannel 及相关定义 - 移除三条充值路由;adminfinance 财务统计口径只统计 order_pay - 清理充值相关测试用例 前端: - 移除充值 API、WalletView 充值面板/弹窗、admin 充值标签与筛选 - 保留钱包余额、流水、提现等核心能力 go build/vet 与 vue-tsc typecheck 均通过。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
45 lines
1.2 KiB
Go
45 lines
1.2 KiB
Go
package wallet
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"testing"
|
|
)
|
|
|
|
// TestServiceAccountWithNilRepo 测试依赖检查
|
|
func TestServiceAccountWithNilRepo(t *testing.T) {
|
|
svc := &Service{repo: nil}
|
|
ctx := context.Background()
|
|
_, err := svc.Account(ctx, 1)
|
|
if !errors.Is(err, ErrDependencyUnavailable) {
|
|
t.Fatalf("Account() error = %v, want ErrDependencyUnavailable", err)
|
|
}
|
|
}
|
|
|
|
func TestServiceLedgerWithNilRepo(t *testing.T) {
|
|
svc := &Service{repo: nil}
|
|
ctx := context.Background()
|
|
_, err := svc.Ledger(ctx, 1, 1, 10)
|
|
if !errors.Is(err, ErrDependencyUnavailable) {
|
|
t.Fatalf("Ledger() error = %v, want ErrDependencyUnavailable", err)
|
|
}
|
|
}
|
|
|
|
func TestServiceWithdrawIsPending(t *testing.T) {
|
|
svc := &Service{repo: &Repository{}}
|
|
ctx := context.Background()
|
|
_, err := svc.Withdraw(ctx, 1, WithdrawRequest{AmountCent: 100})
|
|
if !errors.Is(err, ErrFeaturePending) {
|
|
t.Fatalf("Withdraw() error = %v, want ErrFeaturePending", err)
|
|
}
|
|
}
|
|
|
|
func TestServiceAdminLedgerWithNilRepo(t *testing.T) {
|
|
svc := &Service{repo: nil}
|
|
ctx := context.Background()
|
|
_, err := svc.AdminLedger(ctx, AdminLedgerQuery{Page: 1, PageSize: 10})
|
|
if !errors.Is(err, ErrDependencyUnavailable) {
|
|
t.Fatalf("AdminLedger() error = %v, want ErrDependencyUnavailable", err)
|
|
}
|
|
}
|