google / google/s2-geometry-library-java
S2CellUnionTests Line 258-260 should be removed.
- Ngôn ngữ chính
- Java
- Star
- 587
- Fork
- 231
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
```
Line 258-260 as below should be removed:
if (!u.contains(xId)) {
u.getIntersection(xCells, yId);
}
It is extraneous and guarantees the subsequent assertTrue(u.contains(xId)); on
line 261 will always work even though this is the line that is being ultimately
tested.
```
Original issue reported on code.google.com by `uvico...@gmail.com` on 9 Jul 2013 at 1:39
Hướng dẫn đóng góp
Hướng nghiên cứu
Mở S2CellUnionTests và kiểm tra các dòng 258-261, tập trung vào cách assertion hiện đang được làm cho có thể đạt tới. Xác nhận rằng test vẫn kiểm tra hành vi dự kiến mà không có block thừa, sau đó chạy các test S2CellUnion liên quan và xác minh rằng assertion vẫn có ý nghĩa.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- java
- Lĩnh vực
- testing
- Loại issue
- Tái cấu trúc
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100