Decouple testing of `_conditions.py` from the testing of the public API
Chưa có ai nhận issue này.
Đá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
- 45/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- python
- Lĩnh vực
- testing-qa
Hướng nghiên cứu
Bắt đầu bằng cách đọc _conditions.py và tests/test_joined_equal_unequal.py, đặc biệt là bài kiểm thử được tham số hóa quanh dòng 284. Xem lại các bài kiểm thử API công khai khác để tìm phần bao phủ tolerance tương tự, sau đó tách các kiểm tra ở cấp condition khỏi hành vi dành riêng cho API. Công việc được xem là hoàn tất khi các bài kiểm thử API công khai không còn kiểm thử trùng lặp những phần do _conditions.py đảm nhiệm và phần bao phủ vẫn được duy trì.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
From @EgeKaraismailogluQC:
We are currenly conflating the testing of some functionality defined in _conditions.py with the testing of the public API. For example this test which is meant to test the correctness of joined_equal_unequal is parameterized to test that comparisons with relative/absolute tolerance work as expected.
It would be cleaner if this test, and other tests for the public API assume that the methods defined in _conditions.py are implemented correctly and instead focused on testing only the functionality related to their respective API methods. This way, we can get rid of duplicate testing.
- Ngôn ngữ chính
- Python
- Star
- 47
- Fork
- 1
- Merge trung bình
- 1 ngày 10 giờ
- Pull request đã merge (30 ngày)
- 7
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Quantco/diffly
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
Refactor test setup Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
Tất cả issue của Quantco/diffly
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
stephrobert/dsoxlab#238 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
sublimehq/package_control#1780 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
nwg-piotr/nwg-displays#145 ·