Task #96
closedTask #52: Coding
Coding-Update Unit testcode For Facility Screen
Added by dang dao 5 months ago. Updated 5 months ago.
100%
Description
3 screen
-facility list
-facility detail
-facility New/Update
Files
| UnitTestGuide.md (7.93 KB) UnitTestGuide.md | Nghia Nguyen, 07/17/2025 04:16 AM |
Updated by Nghia Nguyen 5 months ago
- File UnitTestGuide.md UnitTestGuide.md added
@duy có file md nói về unit test trong source
Updated by Nghia Nguyen 5 months ago
- File Screenshot 2025-07-17 at 8.20.57 PM.png added
Trên server staging của facility owner chạy unit test ko có issue
Updated by Anonymous 5 months ago
- % Done changed from 50 to 100
Hi Nghĩa, bên này tui mới code xong, check giúp tui lun nhen
https://github.com/clmt-vn/facility-owner/pull/6/
Updated by Nghia Nguyen 5 months ago
- Assignee changed from Nghia Nguyen to Anonymous
hi @duy, ông xem lại có mấy chỗ ông đang comment á, mấy dòng đó có xài nữa ko. Ko xài thì mình bỏ luôn nha, tui thấy có dòng đang comment // ... nữa á .
ngoài ra còn có chỗ đang test kiểm tra trùng facility name và place. Logic này mình sẽ kiểm tra bên place nên facility mình ko cần kiểm tra cái này nha.
t có comment trong PR nè, Duy xem nha
https://github.com/clmt-vn/facility-owner/pull/6#pullrequestreview-3041820985
Updated by Anonymous 5 months ago
Hi Nghĩa, tui vừa update lại follow theo comment
- fix conflict luôn rồi á
- chỗ function test_既に登録されている施設名・実施場所の組み合わせは登録禁止() tui bỏ là do chỗ này trước logic là truyền name và place vào để test check trùng 2 trường này rồi thả lỗi 409 => giờ không có place nữa thì xóa luôn nhen
Updated by Nghia Nguyen 5 months ago
@duy fix luôn conflict trên https://github.com/clmt-vn/facility-owner/pull/5 này nha
Updated by Nghia Nguyen 5 months ago
- File deleted (
Screenshot 2025-07-17 at 8.20.57 PM.png)
Updated by Nghia Nguyen 5 months ago
- Assignee changed from Nghia Nguyen to dang dao