search

Game-as-a-Service / kuhhandel

幕後交易
2 stars 2 forks source link

codeReview:BDD:接受出價 #5

Closed r-jen closed 3 months ago

r-jen commented 3 months ago
  1. 原掛在 package repository 底下皆為角色身分，掛成跟資料庫溝通用的 repository 怪怪的，暫移定義為 interface ，未確定身分的實作方式，可以討論
  2. 原來的寫法 acceptBid 是透過 then關鍵字觸發的，這部分錯了，可以參考改寫後的 BDD 怎麼 invoke acceptBid，也可以參考範例 BDDMockitoUnitTest 裡的 givenInvalidContactName_whenSearch_thenRetunNull 寫法
  3. 原來的寫法 respondentCards.remove(card) 、respondentHandCard 的操作，是卡片重新分配的邏輯，我沒有規劃放在acceptBid裡面，我會提出來寫成另一個 Story，這樣不管幕後交易在哪個步驟結束，都可以使用該 story 的程式邏輯進行分配卡片資源
  4. then(respondent).should().changeHandCard(respondentHandCard); 原來有類似這種句子，這寫法意思是檢查 該mock物件的method 有沒有被觸發，所以changeHandCard必須在 acceptBid裡面，不然會報沒觸發method的錯誤。
chil2020 commented 3 months ago

GameStatus 好長, 假日再來討論一下