search

Ptt-official-app / Ptt-backend

PTT APP 的後端
BSD 3-Clause "New" or "Revised" License
208 stars 67 forks source link

forward article repository #186

Closed RicoToothless closed 3 years ago

RicoToothless commented 3 years ago

👏 解決掉的 issue / Resolved Issues

📝 相關的 issue / Related Issues

RicoToothless commented 3 years ago

新手提問三個問題：

  1. 請教為什麼 lint error repository.Repository missing method ForwardArticleToBoard？
  2. 第一次寫 Go，是照前面的 PR 依樣畫葫蘆出來的，有沒有遺漏的地方？
  3. 回應文章跟轉錄看板文章似乎有點像。 我實際去看 PTT， 回應文章 (y) 的確有分到看板跟信箱； 轉錄 (^X) 似乎只有轉到別的看板而已。 所以實作上，回應文章到看板跟轉錄是一樣的東西囉？
nickyanggg commented 3 years ago
  1. 請教為什麼 lint error repository.Repository missing method ForwardArticleToBoard？

因為 usecase/mock_test.go 中的 MockRepository 實作 repository.Repository，所以 MockRepository 也必須實作 Repository 定義的所有函式， 之前寫的人都是寫在 usecase/board_mock_test.go 和 usecase/user_mock_test.go 中

y2468101216 commented 3 years ago
  1. 第一次寫 Go，是照前面的 PR 依樣畫葫蘆出來的，有沒有遺漏的地方？

可以先跑 go build 看能不能 build 起來，build 不過肯定沒寫對 再來是跑 go test ./... 跑不過的肯定有問題 最後是跑 ci 跑不過的肯定有問題

RicoToothless commented 3 years ago

因為 usecase/mock_test.go 中的 MockRepository 實作 repository.Repository，所以 MockRepository 也必須實作 Repository 定義的所有函式， 之前寫的人都是寫在 usecase/board_mock_test.go 和 usecase/user_mock_test.go 中

我看了一下，所以我這個似乎要多寫一個 file 為 usecase/article_mock_test.go 囉？

y2468101216 commented 3 years ago

因為 usecase/mock_test.go 中的 MockRepository 實作 repository.Repository，所以 MockRepository 也必須實作 Repository 定義的所有函式， 之前寫的人都是寫在 usecase/board_mock_test.go 和 usecase/user_mock_test.go 中

我看了一下，所以我這個似乎要多寫一個 file 為 usecase/article_mock_test.go 囉？

寫在 article_test.go 裡就好

codecov-commenter commented 3 years ago

Codecov Report

Merging #186 (022321b) into development (f1d627c) will decrease coverage by 0.16%. The diff coverage is 20.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #186      +/-   ##
===============================================
- Coverage        46.25%   46.09%   -0.17%     
===============================================
  Files               25       25              
  Lines             1321     1330       +9     
===============================================
+ Hits               611      613       +2     
- Misses             623      630       +7     
  Partials            87       87
Impacted Files Coverage Δ
internal/repository/article.go 0.00% <0.00%> (ø)
internal/repository/repository.go 0.00% <ø> (ø)
internal/usecase/article.go 59.09% <100.00%> (+6.70%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f1d627c...022321b. Read the comment docs.

RicoToothless commented 3 years ago

我後來還是新增了 usecase/article_mock_test.go file

y2468101216 commented 3 years ago

fix by #203