Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

S2CellUnionTests Line 258-260 should be removed.

Đang mở
  • Có gì không ổn?
#1 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

@ThomasThelen đang làm issue này rồi.

Từ ngày 7/4/2026.

  • #54 của @ThomasThelen — đang mở

Đá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
35/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ệ
java
Lĩnh vực
testing

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.

Mô tả

auto-migrated Priority-Medium Type-Defect 
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 [email protected] on 9 Jul 2013 at 1:39

Ngôn ngữ chính
Java
Star
587
Fork
232
Không có pull request nào được merge trong 30 ngày

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 google/s2-geometry-library-java

Tất cả issue của google/s2-geometry-library-java

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.