Project

General

Profile

Actions

Task #96

closed

Task #52: Coding

Coding-Update Unit testcode For Facility Screen

Added by dang dao 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Start date:
07/17/2025
Due date:
07/22/2025
% Done:

100%

Estimated time:

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
Actions #1

Updated by Anonymous 5 months ago

  • Due date set to 07/17/2025
  • Status changed from New to In Progress
Actions #2

Updated by Nghia Nguyen 5 months ago

@duy có file md nói về unit test trong source

Actions #3

Updated by Anonymous 5 months ago

  • Due date changed from 07/17/2025 to 07/18/2025
  • % Done changed from 0 to 50
Actions #4

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

Actions #5

Updated by dang dao 5 months ago

  • Subject changed from Update Unit testcode For Facility Screen to Coding-Update Unit testcode For Facility Screen
Actions #6

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/

Actions #7

Updated by Anonymous 5 months ago

  • Assignee changed from Anonymous to Nghia Nguyen
Actions #8

Updated by dang dao 5 months ago

  • Due date changed from 07/18/2025 to 07/22/2025
Actions #9

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

Actions #10

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

https://github.com/clmt-vn/facility-owner/pull/6

Actions #11

Updated by Anonymous 5 months ago

  • Assignee changed from Anonymous to Nghia Nguyen
Actions #12

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

Actions #13

Updated by Nghia Nguyen 5 months ago

code unit test đã test ok, đã merged

Actions #14

Updated by Nghia Nguyen 5 months ago

  • File deleted (Screenshot 2025-07-17 at 8.20.57 PM.png)
Actions #15

Updated by Nghia Nguyen 5 months ago

  • Assignee changed from Nghia Nguyen to dang dao
Actions #16

Updated by dang dao 5 months ago

  • Status changed from In Progress to Closed
Actions

Also available in: Atom PDF