Re: [討論] Unit test 的撰寫請益

作者: HZYSoft (PCMan)   2022-11-10 23:21:01
※ 引述《shane87123 (陽光大肥宅)》之銘言:
: 先說我對 Unit test 的看法:測試單元(可能是 function)的邏輯是否正確
: 好,進入正題
: 小弟最近剛工作,稍微讀了一下負責的 project 的程式碼後,
: 要開始開發 Unit test。
: 現況是,各個 file (.c) dependency 很重,
: 常常會有一份 code 內其實呼叫了很多別份 code 的 function,
: 舉例來說
: A() {
: B();
: C();
: if (check)
: D();
: }
: 族繁不及備載,
: 而我目前設計有兩個方向,
: 1.
: 將 B() C() D() 全部 fake ,單純去測試 A() 的邏輯是否正確
: 這樣做感覺上會比較單純,一個 test case 只去 test A(),
: 而且不需要去 include B() C() D() 的 header,
: 這樣一來 build 起來也比較容易,因為 include 那些 header 又會 dependency 到其他檔
: 情況會非常複雜
: 缺點是 coverage 比較差,B() C() D()要額外去寫 test case
先說結論,先都不要寫。
Legacy system 要先補大範圍的 integration test,確定整體的行為是對的。
如果 code 沒有要再改,不用補細部 unit tests。
原因是因為,原本 API 可能因為設計不良,導致無法寫 unit test
得先 refactor 才有辦法讓它變成 testable,這情況就要先 refactor 再補 UT
而 integration test 會一定程度減少 refactor 造成 regression bugs。
再來正名運動一下,fake/mock/stub 都是 test double,但用途不一樣
https://martinfowler.com/articles/mocksArentStubs.html
fake 仍然是一個大致完整的實作,只是比較簡化。
例如本來是 on-disk 的 key-value database,fake 就可以用 in-memory 的一個
dictionary 物件換掉,簡化許多但是 API 有提供 "一樣的行為 (key 的讀寫)"
stub 則是多半只會 return 固定的 constant,本身沒實作行為
用 test double 主要好處是減少 dependency,然後測試速度會快。
測試大規模系統,如果讀寫檔案,讀寫資料庫都是真的去讀,那就牽扯到跨系統整合
會花超多時間跑,也比較是 integration test
Unit test 的目標要能快速給予回饋,所以很強調執行速度。用 test double 換掉
同時能提升 isolation,也避免一些意外狀況造成測試失敗。
例如程式邏輯明明沒錯,卻因為外部因素而失敗跑不過,
所以 unit test 的書上多推廣用 test double。
但實務上,持續維護 test double 和真實物件的行為一致,是一件高成本而且困難的事
如果你改變了真實物件的行為,所有有用 test double 的地方都要重新改寫
而且 fake 物件實作有是可能會錯,那反而會製造更多問題
所以 To fake or not to fake? That is the question.
後來有些書上就提倡 prefer real object,能用真的就盡量不要用假的。
根據專案的狀況,你可以選擇最適合的方式,沒有絕對正確答案
然後目標不是高 coverage。目標是最 business critical 的 path 都有測試到。
coverage 要高很簡單,全部 function 呼叫一次,然後不要檢查任何東西
coverage 就會 100%,但什麼也沒測試到。
只要關鍵行為都測試到了,不用太糾結 coverage 數字高不高
: 2.
: 直接把他們 include 進來,build failed 就 include,直到 build 過為止
: 這樣的好處是不用去實作 B() C() D() 的 fake,
: 但就會讓整個 unit test 的 dependency 很重
這有時候是合理的作法。
Unit test 是依照"行為"拆分,不是依照 class/function 拆分。
不是一個 function 就對應一個 test,而是以"user 可見的行為" 來拆。
一個功能/行為常常需要多個 functions 組合,這些 private function 都是"實作細節"
是沒有必要測試的,你只需要測試 public interface.
同理,一個功能由 A/B/C 三部分組成,A 呼叫 B 和 C,但實際上對外使用者只用 A
那 B/C 可以視為 A 的實作細節,這時候可以只針對 A 做 test
一個 class 的 public API 可以是另外一個 class 的 implementation detail
所以不是一個 class 就要一個 test,實作細節不用測,要測試大的行為。
: 個人偏向1.,畢竟 unit test 就是去測試 function 的邏輯性,
: 在其他 function 對測試 function 沒有 side effect 的情況下(如不會改變某變數的值?
: 將他們 fake 掉而只是單純的去 test 該 function 而已
: 但我第一次接觸,不太知道何時應該去 fake (或 mock) 一個 function QQ
: 我只是有這兩種想法,兩個其實天差地遠XDD
:
作者: Hsins (翔)   2022-11-10 23:24:00
Sent from PCMan on PCMan's PC by the author of PCMan -PCMan.
作者: Sean0428 (我要賺大錢)   2022-11-10 23:28:00
Good
作者: wulouise (在線上!=在電腦前)   2022-11-10 23:32:00
不過實作細節要分到多細還是看個人考量..不然整個chrome只測UI好像也符合這個分法(?)
作者: peyton87 (小培培)   2022-11-10 23:34:00
推legacy系統要先大範圍integration test。之前摸索時先搞了些mock物件,弄老半天只測一小塊。直接做必要的fake然後大塊功能的integration test,至少能測。因為先有integration test,才成功做了些重構。
作者: drajan (EasoN)   2022-11-11 00:25:00
同意在缺乏好的模組化跟測試的前提下 整合測試是最優先的確保最大範圍的抽象行為是正確的再去改內部的程式
作者: CKNTUErnie (德田田馥甄)   2022-11-11 00:27:00
作者: noahleft (NoahLeft)   2022-11-11 01:45:00
作者: yuinami (yuinami)   2022-11-11 02:03:00
作者: whatzup1124 (我是幹嘛)   2022-11-11 02:23:00
作者: bnd0327 (阿噗噗)   2022-11-11 08:21:00
推推
作者: lchcoding   2022-11-11 08:40:00
拜讀
作者: CRPKT (crpkt)   2022-11-11 08:40:00
這篇每一段的概念都值得推一次
作者: sck921 (The Fate)   2022-11-11 10:57:00
作者: alan5 (小安)   2022-11-11 10:58:00
還在那邊想是誰觀念這麼正確...
作者: ntps60803orz (ntps60803)   2022-11-11 11:25:00
為什麼全部function呼叫一次就可以達到100% coverage?不考慮條件式嗎?
作者: rizman28 (栗子)   2022-11-11 12:03:00
作者: ManOfSteel (Man Of Steel)   2022-11-11 12:04:00
作者: Walkers (walkers)   2022-11-11 12:19:00
先推
作者: newhandfun (新手方)   2022-11-11 12:40:00
我們也是先整合再單元,原因就是覺得之後可能會重構
作者: richer6605 (Rhapsody)   2022-11-11 12:44:00
作者: lukelan (藍丁丁)   2022-11-11 13:54:00
pcman 大神推推
作者: scottxxx666 (高高)   2022-11-11 14:09:00
作者: shibin (喜餅)   2022-11-11 15:34:00
作者: sky80420 (澤西哥)   2022-11-11 17:35:00
先推再看
作者: d07880201 (西瓜瓜瓜瓜)   2022-11-11 19:43:00
作者: bewitchsky (Shopping)   2022-11-11 22:22:00
作者: jskblack (天空茶)   2022-11-11 22:34:00
推受用
作者: devilkool (對貓毛過敏的貓控)   2022-11-12 00:27:00
作者: yupog2003 (屁股)   2022-11-12 10:13:00
我們公司完全照著這篇文章的觀念在寫unit test
作者: a4782887   2022-11-12 16:45:00
這篇寫的很好 感謝分享
作者: jasonwung (路人JJ)   2022-11-13 01:18:00
作者: remember69 (玻璃心先生)   2022-11-13 14:49:00
受用 推
作者: uziel (= ̄ω ̄=)   2022-11-15 07:35:00
同意,要測就測抽象行為
作者: zapion (SZ)   2022-11-16 09:47:00
推這篇
作者: streakray (條紋衣boy)   2022-11-22 12:10:00
作者: judy2r3 (小穎≧﹋≦)   2022-11-28 04:46:00
實用正確 推
作者: slowwalker (慢行者)   2022-12-01 05:06:00

Links booklink

Contact Us: admin [ a t ] ucptt.com