Some unit tests don't make sense

Đang mở
  • Có gì không ổn?
#108 5 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
25/100
Loại issue
Tài liệu
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
javascript, node.js
Lĩnh vực
documentation, testing

Hướng nghiên cứu

Bắt đầu với các trường hợp kiểm thử đơn vị được báo cáo là 223 và 303, sau đó đọc hành vi so sánh bằng sâu mà các kiểm thử đó thực hiện. Giải thích vì sao các kỳ vọng hiện tại là có chủ đích, hoặc xác định một điểm không khớp cụ thể cần được thay đổi; công việc hoàn tất khi kết quả kiểm thử và ngữ nghĩa của phép so sánh bằng được ghi lại rõ ràng.

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

Mô tả

Trawling thru the unit tests, I've noticed some failed tests that I really think should pass.
For example:
not ok 223 empty buffer and empty array are not equal
I, would honestly expect and empty buffer and empty array to NOT be equal. One holds arbitrary data and one holds bytes. So why then, are them not being equal a fail?
Also:
not ok 303 Date and RegExp are not equal
I cannot see any scenario where a Date and a RegExp would be any type of equal, them being 2 completely different types with completely different intended functionality. So then why is them not being equal a failed test?

I want to adopt this library, but I need to ensure I won't have any... Surprises.

Can someone explain these test results and maybe convince me to integrate this library?

Ngôn ngữ chính
JavaScript
Star
792
Fork
107
Merge trung bình
9 ngày 20 giờ
Pull request đã merge (30 ngày)
1

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 inspect-js/node-deep-equal

Tất cả issue của inspect-js/node-deep-equal

Issue tương tự

Thêm issue về JavaScript

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.