kenyiu / newsdiffHK

newsdiff for HK
4 stars 1 forks source link

Unit test 及新聞來源修正 #11

Closed yookoala closed 9 years ago

yookoala commented 9 years ago

Crawler* 類別,頗易受新聞網站更新所害,需要維護。但到需要維護時,由於舊版網站已經不存在,難以比對判定是否有結構改動,或者改動在甚麼地方出現。另外在改動時,在原有的程式架構下頗難自動測試,因此想修改程式架構,持續測試目前的 Crawler* 類別。

  1. 將原有程式架構,改得較易做 unit test
  2. 在各大新聞來源下載一些範例內容,以方便進行 unit test
  3. 為所有現存的 Crawler 編寫對應的 unit test
  4. 加入 Travis CI 設定檔,讓 Travis 可以自動對每一個 push 進行 unit test

另外有一些零星的改善,包括︰

  1. 將新聞來源的定義統一到一個地方去,方便新增來源(這方便我們在 Code4HK 的工作)
  2. 改用 git submodule 連結 url-normalizer.js,讓源碼控制較為靈活。
yookoala commented 9 years ago

放錯地方,其實是想送去 upstream 的,已經加入到那邊的 pull request 19