google / google/s2-geometry-library-java

S2CellUnionTests Line 258-260 should be removed.

Đang mở
#1 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
auto-migrated Priority-Medium Type-Defect
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

Mở 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

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.