Phương pháp xác nhận JUnit nên được diễn giải trong tích cực hoặc tiêu cực?

Question

Tôi có nên viết
assertTrue("User logged in", user.isLoggedIn());
hoặc
assertTrue("User is not logged in", user.isLoggedIn());

Cựu cung cấp đọc tốt hơn bên trong các tập tin nguồn:
"Tôi khẳng định rằng sau đây là đúng: Người dùng đăng nhập."

Các thông báo lỗi có thể được đọc cả hai cách: "Các lỗi được mà người dùng đang đăng nhập"
java.lang.AssertionError: User logged in
"Có một lỗi trong việc khẳng định mà người dùng đang đăng nhập"

tài liệu JUnit không cung cấp một hướng dẫn rõ ràng mà nó nên được, ngoại trừ nó là
"thông điệp xác định cho {@link AssertionError}",
Và trong cả hai trường hợp, các văn bản xác định thử nghiệm hạnh phúc chạy.

Cách sử dụng phổ biến là gì?

Answer 1

Làm thế nào về:

assertTrue("User should be logged in", user.isLoggedIn()); 

trình cả hai cách.

Answer 2

Vâng, bạn cũng có thể tuyên bố giả định của bạn, và sau đó giả định không giữ. Giống như vậy:

assertTrue("Expected user to be logged it, and wasn't", user.isLoggedIn()); 

Làm cho các thông điệp rõ ràng hơn, nhưng còn phải nhập và đọc.

Answer 3

Bạn nên bao gồm cả hai trường hợp. Bạn có trường hợp kiểm tra tốt hơn khi bạn triangulate xác nhận của bạn.

Answer 4

Để tránh câu hỏi đó, tôi ngày càng có xu hướng sử dụng assertThat thay vì các phương pháp * khẳng định "cấp thấp". Thật vậy, như this article giải thích, khẳng định rằng sẽ cung cấp cho bạn một thông báo lỗi rất rõ ràng trong trường hợp thất bại.

Answer 5

Bạn có thể sử dụng:

assertTrue("Test if user is logged in", user.isLoggedIn()); 

Khi bạn làm điều này, bạn đang xác minh rằng user.isLoggedIn() là đúng, bạn có thể không thực sự nói rằng người dùng đang đăng nhập hay không, bạn vẫn chưa biết, bạn chỉ đang thử nghiệm nó.

Answer 6

Thú vị, tôi sẽ sử dụng:

assertTrue("user should be logged in", user.isLoggedIn()); 

mà nói với tôi gì được mong đợi nhà nước của khẳng định này.

Tôi nghĩ lựa chọn tốt nhất là sự lựa chọn của bạn.

Answer 7

Hãy nghiêm túc tích cực trong thông điệp khẳng định của bạn

Sử dụng tích cực văn bản khẳng định như trong ví dụ đầu tiên của bạn, hoặc như:

assertTrue("User is logged in", user.isLoggedIn()); 

Lý do là:

• khẳng định dương tính là ngắn hơn
• có một điều kiện được xác nhận mà bạn đang kiểm tra và nhiều lý do có thể xảy ra, tại sao nó lại sai. Không cố gắng để phát hiện nguyên nhân, chỉ cần nhà nước, những gì khẳng định đã thất bại.
• nó dễ đọc hơn trong mã của bạn. Nó thường được khuyến khích để mã trong các biểu thức tích cực, mà tiết kiệm vài negations của các điều kiện trong tâm trí của người đọc
• nó vẫn có thể đọc được trong các dấu vết lỗi, mà không được hiểu bởi người dùng thông thường, nhưng bởi một lập trình viên, những người sẽ kết thúc trong mã anyway. Và thậm chí sysadmin, những người sẽ không có quyền truy cập vào các mã sẽ cung cấp cho tác giả với thông báo lỗi hoàn chỉnh và lập trình viên sẽ hiểu, nó đến từ một khẳng định.

Cố gắng cung cấp "tất cả thông tin theo ngữ cảnh" trong thông báo khẳng định không cải thiện tình hình, thay vào đó tạo ra sự lộn xộn thông tin.

Bạn biết đấy, trình lập trình tốt gỡ lỗi mã và cung cấp mã làm việc và ngắn hơn.

Sử dụng thông điệp khẳng định tích cực làm điều đầu tiên để thực hiện theo hướng này.

Một hướng khác - vá mã với nhiều thứ không cần thiết hơn là mở đường đến địa ngục lập trình.