パッケージ構成の変更

[ras0q]

プロジェクト内の一連の処理をinternalパッケージ以下にまとめた
util->pkgs,interfaces/handler->handlerなどの変更も入れている
util(pkgs)はinternalの外に出してもいいがtraPortfolio以外のパッケージから使われることもないのでinternal以下に置いた

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.46%. Comparing base (b0977ea) to head (d5e27ce).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #705      +/-   ##
==========================================
+ Coverage   73.49%   74.46%   +0.97%     
==========================================
  Files          54       52       -2     
  Lines        4218     4163      -55     
==========================================
  Hits         3100     3100              
+ Misses        965      910      -55     
  Partials      153      153              

Flag	Coverage Δ
integration	74.46% <ø> (+0.97%)	⬆️
unit	74.46% <ø> (+0.97%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[H1rono]

良さそうなんですが、integration_testsだけinternalの外にあるのはどうしてですか？これも動かしちゃっていい気がしますが

[ras0q]

分けてるのをよく見るくらいの理由しか出せないな、、

https://github.com/moby/moby
https://github.com/googleapis/google-api-go-client

[ras0q]

利点としてはgo test ./internalとgo test ./integration_testsでテストの実行を分けられるとかがある (internalの中に入れると変更前のようにgo listをごにょごにょする必要がある)

[ras0q]

とりあえずマージ