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

Follow-up: a6 test / skills / docs review findings

Đang mở
  • Có gì không ổn?
#14 4 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
45/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
go
Lĩnh vực
cli, testing

Hướng nghiên cứu

Bắt đầu với TestSkillPluginSkywalking và các bài kiểm thử E2E hiện có của skills, sau đó xem lại báo cáo coverage và các module dùng chung được nêu trong issue: selector, tableprinter, root và configutil. Bổ sung coverage cho các capabilities có điều kiện, khả năng tương thích của tên container, các nhãn Global Rule không được hỗ trợ, các module dùng chung và các kịch bản kết hợp; hoàn thành nghĩa là các trường hợp được theo dõi riêng biệt và các bài kiểm thử cùng tài liệu vẫn nhất quán với nhau.

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

Mô tả

Background

Before putting a6 into real use, we completed a full review and local rerun of the current test system, skills, and documentation.

Detailed Feishu report:

Current Conclusions

  • The baseline CI test suite passes.
  • With the current CI E2E workflow deployment model, the a6 E2E suite can also be reproduced successfully in a local environment.
  • skills metadata validation passes.
  • After running the skills E2E suite locally against a real environment, all cases pass except skywalking, which is skipped because the required conditional plugin is missing.
  • The current overall code coverage is approximately 52.4%.

This indicates that a6 already has a relatively complete automation baseline, but there are still gaps before it can be considered ready for “100% coverage / full pre-release validation”.

Follow-up Items

P0
  • Clarify the conditional capability boundary for the skywalking skill
    • TestSkillPluginSkywalking is currently skipped with the current image
    • We need to make it explicit that this is a conditional skip caused by a missing plugin, not a fully covered and passing case
P1

  • Add compatibility tests for debug logs container naming

    • The current setup passes with the CI container name apisix
    • However, the test currently assumes a hard-coded container name, which can easily lead to mismatches in local reproduction

  • Add capability-boundary coverage for Global Rule labels

    • The related case is currently skipped because APISIX does not support it

  • Add tests for shared modules and low-coverage modules

    • Such as selector, tableprinter, root, configutil, and similar modules

  • Add more combined scenario coverage

    • service + route + plugin-config
    • Real traffic verification for multi-node upstreams
    • More non-happy-path and invalid-parameter branches
P2
  • Continue raising low-coverage packages one by one based on the coverage report by adding more unit tests

Suggested Outputs

  • Track conditional-capability cases separately from real passing cases
  • Add another round of test supplementation using semantic scenarios as the unit of coverage
  • Keep tests and documentation aligned in follow-up PRs
Ngôn ngữ chính
Go
Star
1
Fork
2
Merge trung bình
24 phút
Pull request đã merge (30 ngày)
1

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

  • Không có Dockerfile hay tệp Docker Compose
  • Có mẫu pull request
  • Không có hướng dẫn đóng góp

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 api7/a6

Tất cả issue của api7/a6

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.