test(flagd): drop the per-scenario /stop in e2e teardown once the testbed includes flagd-testbed#396

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,891 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
68/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java
Lĩnh vực
testing

Hướng nghiên cứu

Bắt đầu tại providers/flagd/src/test/java/.../e2e/steps/ProviderSteps.java và đọc ProviderSteps.tearDown để tìm khối POST /stop, cùng với ContainerPool.release và phần hoán đổi provider NoOp phải được giữ lại. Trước tiên hãy xác nhận dependency flagd-testbed đã được bump lên một release chứa open-feature/flagd-testbed#396 (kiểm tra phiên bản testbed trong các file build của provider flagd), vì việc gỡ /stop trước đó sẽ không mang lại gì. Xong nghĩa là khối /stop đã biến mất khỏi tearDown trong khi ContainerPool.release và phần hoán đổi provider vẫn còn, và toàn bộ bộ e2e (in-process, RPC, file providers) vẫn pass.

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

Mô tả

Problem

ProviderSteps.tearDown (providers/flagd/src/test/java/.../e2e/steps/ProviderSteps.java) POSTs /stop to the launchpad after every scenario. That makes every /start a cold flagd start.

open-feature/flagd-testbed#396 makes /start reuse the running flagd and restore the baseline flag state in place. Isolation no longer needs the /stop, and keeping it means the suite never gets the speedup.

Proposal

After bumping the testbed to a release that includes open-feature/flagd-testbed#396, remove the /stop block from tearDown. Keep ContainerPool.release and the NoOp provider swap.

Evidence

Tested locally with the /stop removed, against an image built from flagd-testbed#396 and one built from testbed main. All 590 tests (in-process 224, RPC 214, file 152) passed in every run. Wall time (in-process / RPC / file):

with /stop without /stop
flagd-testbed#396 102-107s / 73-79s / 48-65s 96s / 66s / 41s
testbed main 100s / 75s / 50s 120s / 79s / 49s

The machine was shared with other test runs, so treat the times as indicative. Without flagd-testbed#396, dropping /stop gains nothing, which is why this should wait for the bump.

Ngôn ngữ chính
Java
Star
50
Fork
87
Merge trung bình
1 ngày 14 giờ
Pull request đã merge (30 ngày)
53

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 open-feature/java-sdk-contrib

Tất cả issue của open-feature/java-sdk-contrib

Issue tương tự

Thêm issue về Java

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.