Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

abctl: the casefold breakdown test's second assertion is unreachable on its own fixture

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,179 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
86/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go
Lĩnh vực
cli, testing-qa

Hướng nghiên cứu

Đọc cmd/abctl/cmd_cost_test.go và kiểm tra TestRunCost_ByCurrencyMatchesAUnitSpelledADifferentWay, đặc biệt là fixture Credits chỉ có một entry và cả hai assertion. Xác nhận assertion nào không thể đạt tới, sau đó làm cho mệnh đề duy nhất của test trở nên rõ ràng bằng cách thêm một fixture thực sự hỗn hợp hoặc xóa assertion thừa; test phải bao quát chính xác hành vi casefold mà không ngụ ý một nhánh chưa được kiểm thử.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Found by the round-3 strict review of #1153, in a test that PR's round 2 added.

cmd/abctl/cmd_cost_test.go, TestRunCost_ByCurrencyMatchesAUnitSpelledADifferentWay:

if strings.Contains(row, mixedCostCell) {
    t.Errorf("a real configured unit was withheld as unrecognised:\n%s", row)
}

The fixture carries "currencies":["Credits"] — one entry — so windowUnit returns labelled=true and writeCostBreakdown's default: cost = mixedCostCell arm is unreachable for a priced row. The assertion cannot fire on this input, and its stated reason ("withheld as unrecognised") describes a path the fixture does not reach.

The test's first assertion is live and does pin the property that matters (a unit spelled differently from the reported set keeps its label, i.e. isReportedUnit folds case). Only the negative twin is dead.

Fix is either a second fixture whose window is genuinely mixed, or dropping the assertion and saying in the comment that the single-unit fixture cannot produce (mixed). Prefer whichever keeps the test's one claim legible.

Assisted-By: Claude (Anthropic AI) [email protected]

Ngôn ngữ chính
Go
Star
13
Fork
40
Merge trung bình
12 giờ 10 phút
Pull request đã merge (30 ngày)
191

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của rossoctl/cortex

Tất cả issue của rossoctl/cortex

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.