Add C0 unit test for UsernameValidation#7
Conversation
namnhp-2137
force-pushed
the
C0_unit_test_for_username_valid
branch
from
5f71006 to
2c994d3
Compare
| { | ||
| // TODO: write 6 tests to coverage 100% code | ||
| function generateRandomString($length = 10) { | ||
| return substr(str_shuffle(str_repeat($x = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ', ceil($length/strlen($x)))), 1, $length); |
There was a problem hiding this comment.
Có cần phải hàm phức tạp này không em? Viết sao cho nó vừa đúng case, vừa đơn giản, vừa đảm bảo chạy nhanh
There was a problem hiding this comment.
Em thấy testcase nó cũng như class bình thường thôi ạ, viết sao cho nó thoải mái + reuse cao là được. Ngoài ra nữa thì hàm này sẽ suppỏt cho 1 case là nếu anh change MaxLength và MinLength thì nó vẫn đảm bảo là test case vẫn đúng. Ở đây em ko muốn người ta phải tập trung vào cách define hoặc config mà tập chung vào test thì nó sẽ oke hơn
There was a problem hiding this comment.
Nếu anh viết test case như thế này thì có đơn giản hơn không :D
public function test_IT_not_pass_WHEN_value_exceed_maxlength()
{
$username = str_pad('a', UsernameValidation::MAX_LENGTH + 1, 'b');
$validation = new UsernameValidation;
$this->assertFalse($validation->isValid($username));
}There was a problem hiding this comment.
str_pad('a', UsernameValidation::MAX_LENGTH + 1, 'b');
Oke a, cái này em giờ mới biết dùng được kiểu này :v
để e sửa ạ
| $this->assertEquals('Maximum length is ' . UsernameValidation::MAX_LENGTH, $usernameValidation->getMessage()); | ||
| } | ||
|
|
||
| public function test_username_have_dash_at_begin_or_end_then_false_and_show_message() |
There was a problem hiding this comment.
So với cách đặt tên:
test_IT_not_pass_WHEN_value_has_dash_at_begining
Thì có dễ hiểu hơn không nhỉ? Ở đây IT là class under test => it not pass nghĩa là usernam validation not pass WHEN là điều kiện đầu vào
Khi dùng --testdox ta sẽ có message:
Username Validation (Tests\C0\UsernameValidation)
✔ IT not pass WHEN value exceed maxlength
✔ IT not pass WHEN value has dash at begining
✔ IT not pass WHEN value has multiple dashes
There was a problem hiding this comment.
@tuanpt-0634
VIết như anh cũng được, nói chung tùy vào ngữ cảnh. Nhưng trên thực tế thì sử dụng từ pass hoặc từ it thì nó khá là ít ngữ cảnh để dùng và thực ra nó khá là chung chung. Cái ở đây em muốn mọi người nhìn vào có thể hiểu ngay lập tức function này test gì chứ ko phải là nhìn vào function xong nhìn thêm vào class nữa vì trong trường hợp 1 class nhiều case thì giả sử nhé số case xuống dòng vượt quá chiều cao của màn hình => anh phải khéo lên tìm cái class của nó để xem nó đang test cái gì...
There was a problem hiding this comment.
pass ở đây dùng cho class validation thôi, vì validation thì chỉ có pass hay failed. Còn anh đang muốn suggest chung 1 pattern đặt tên cho test, ko bắt buộc nhưng dễ follow lúc đầu. Ở đây anh nghĩ nên xem class là unit thay vì từng method, khi đó mình sẽ test hành vi của class thay vì test từng method, chẳng hạn có thay đổi tên method nhưng có thể không cần đặt lại tên test case, anh đúc kết từ bài này https://dzone.com/articles/7-popular-unit-test-naming
Should_ExpectedBehavior_When_StateUnderTest() {}
Should_ThrowException_When_AgeLessThan18() {}
Should_FailToWithdrawMoney_ForInvalidAccount() {}
Should_FailToAdmit_IfMandatoryFieldsAreMissing() {}=>
public function test_it_expected_behavior_when_condition() {}There was a problem hiding this comment.
oki anh, thế thì mình nên tổng hợp lại rồi để đó là 1 convention ạ . Kiểu thế thì mọi người cũng dễ follow hơn
There was a problem hiding this comment.
anw nếu mọi người quen rồi thì đặt như thế nào cũng được mà a :))), Freestyle mà ✌️
Miễn sao nó thể hiện được tính chất là test gì ở đâu và kết quả như nào là ổn áp ạ :D
2 participants
Purpose/Notes (*)
C0: Unit testt for class UsernameValidation
#3
Coverage (*)
(PHPUnit coverage screenshot)
Checklist (*)