Deprecate should_not raise_error in MSpec and replaces usages in ruby/spec
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- 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ệ
- ruby
- Lĩnh vực
- testing-qa
Hướng nghiên cứu
Xem lại phần thảo luận của pull request ruby/spec được tham chiếu và tìm trong repository khoảng 176 chỗ sử dụng should_not raise_error. Phân biệt các kiểm tra ngoại lệ hợp lệ với những trường hợp chỉ xác minh rằng không có ngoại lệ xảy ra, sau đó thay thế hoặc loại bỏ các trường hợp sau bằng những kỳ vọng hữu ích. Hoàn thành khi các chỗ sử dụng không cần thiết đã được xử lý và hành vi của MSpec đã được deprecated một cách nhất quán.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
See https://github.com/ruby/spec/pull/753/files#r376792963
should_not raise_error is the same as testing nothing.
Each example already implicitly test no exception is raised.
There might be a few legit cases, but I would expect the vast majority could be removed and replaced with useful expectations actually testing it works as expected.
Currently there are about 176 should_not raise_error in ruby/spec.
- Ngôn ngữ chính
- Ruby
- Star
- 622
- Fork
- 402
- Merge trung bình
- 14 giờ 51 phút
- Pull request đã merge (30 ngày)
- 6
Hướng dẫn đóng góp
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 ruby/spec
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 52/100
-
Improving Marshal specs Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 38/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
ruby/spec#1350 · 3 bình luận · 3 reaction · 1 người được giao ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
TheOdinProject/curriculum#31417 · 2 bình luận ·
-
Allow faraday-http-cache 3.x Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
glossarist/glossarist-ruby#238 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
palladius/rails8-app-on-gcp#145 ·