Adopt shared mapper test data from engine-test-data

Đang mở
#181 0 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
38/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
backend, testing-qa

Hướng nghiên cứu

Bắt đầu với flagsmith/mappers.py quanh dòng 182 và xem các bài kiểm thử mapper hiện có. Xem lại pull request #46 của engine-test-data và mapper_test_cases, sau đó so sánh với các triển khai PHP và .NET được liên kết. Công việc được xem là hoàn tất khi các case dùng chung đều pass và các khóa virtual segment của identity override tuân theo tiêu chuẩn portable JSON cộng SHA256.

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

Mô tả

Mapper tests use inline expected context data rather than shared test cases from engine-test-data repository.

Acceptance criteria

  • mapper_test_cases pass (see Flagsmith/engine-test-data#46)
  • Identity override virtual segment key generation matches cross-SDK portable standard (JSON + SHA256)

Technical context

Current implementation uses non-portable hash:
https://github.com/Flagsmith/flagsmith-python-client/blob/c0d57ec4cd208b79908298e6cf9a3504e5d42fda/flagsmith/mappers.py#L182

Portable standard (JSON + SHA256):

Shared test data: https://github.com/Flagsmith/engine-test-data/pull/46

Related issues:

  • Flagsmith/flagsmith-ruby-client#93
  • Flagsmith/flagsmith-java-client#197
  • Flagsmith/flagsmith-nodejs-client#228
  • Flagsmith/flagsmith-go-client#196
  • Flagsmith/flagsmith-rust-flag-engine#26
  • Flagsmith/flagsmith-dotnet-client#180
Ngôn ngữ chính
Python
Star
24
Fork
12
Merge trung bình
15 giờ 43 phút
Pull request đã merge (30 ngày)
4

Hướng dẫn đóng góp

Mở 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 Flagsmith/flagsmith-python-client

Tất cả issue của Flagsmith/flagsmith-python-client

Issue tương tự

Thêm issue về Python

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.