Add unit tests for applierGroup.Get in Kubernetes executor

Đang mở
  • Có gì không ổn?
#7,003 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 2 ngày

@akshat-lakhera đang làm issue này rồi.

Từ ngày 17/10/2026.

  • #7004 của @siddhiguptas — đã đóng, không merge

Đánh giá

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

Hướng nghiên cứu

Bắt đầu với pkg/app/piped/executor/kubernetes/applier_group.go và kiểm tra applier_group_test.go hiện có trước khi chạy các bài test của package. Triển khai TestApplierGroupGet với các applier được mock, bao quát fallback, định tuyến theo tên provider, việc xây dựng nhiều applier bằng provider-label và các lỗi khi thiếu provider hoặc applier. Hoàn tất khi các trường hợp định tuyến và việc truyền lỗi được bao phủ bằng các unit test chạy thành công.

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

Mô tả

Description

Currently, there is no unit test coverage for the routing logic in the applierGroup.Get method. The production code has a pending comment requesting testing coverage for this component. This issue proposes adding unit tests to verify that the Kubernetes applier is correctly resolved and returned based on matching routes (Kind, Name, Labels, and fallback).

Relevant Code
  • pkg/app/piped/executor/kubernetes/applier_group.go
Proposed Changes
  • Implement TestApplierGroupGet in applier_group_test.go.
  • Cover the following scenarios using mocked appliers:
    • Fallback to defaultApplier when routes are empty or matchers don't align.
    • Successful routing by provider name.
    • Multi-applier construction when routing by provider labels.
    • Error propagation when matched providers or appliers are missing.
Ngôn ngữ chính
Go
Star
1.4k
Fork
391
Merge trung bình
4 ngày 1 giờ
Pull request đã merge (30 ngày)
58

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 pipe-cd/pipecd

Tất cả issue của pipe-cd/pipecd

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.