archlinuxcn / lilac

Lilac is the build bot for archlinuxcn
GNU General Public License v3.0
113 stars 40 forks source link

add some unittests of api.py #126

Closed renyuneyun closed 4 years ago

renyuneyun commented 4 years ago

增加單元測試來輔助開發與debug。

其實是發現api.add_into_array()有一點小問題(特定條件下會出問題),爲了輔助修正,順便增加單元測試。

話說 test/test_lilaclib.py是怎麼用的?需要加到run_test.sh裏麼?

renyuneyun commented 4 years ago

啊,對了:unittest這個模塊好像是python 3.5加入標準庫的。 現在lilac的README說python >= 3.4,和這點有點衝突。

lilydjwg commented 4 years ago

请用 pytest 并且放到单独的文件里。

renyuneyun commented 4 years ago

咦?我還以爲標準庫的更爲推薦呢 ○| ̄|_

放到單獨文件的話,對那兩個內部函數 _xxx() 進行測試難道不會報警告麼?

renyuneyun commented 4 years ago

已完成。 改用 pytest ,刪除沒必要的 run_test.sh。

(仔細一想好像報警告也無所謂…… mypy 又不對 test 目錄執行……)

lilydjwg commented 4 years ago

自带的那个不好用啊。pytest 用着多舒服。